From 366f98dd0dc36fd94bce57a7627a38c7cf91e1ca Mon Sep 17 00:00:00 2001 From: jonahwilliams Date: Tue, 27 Feb 2024 19:58:43 -0800 Subject: [PATCH] [Impeller] remove warning about device transient depth+stencil. --- impeller/entity/entity_pass.cc | 24 +++--------------------- 1 file changed, 3 insertions(+), 21 deletions(-) diff --git a/impeller/entity/entity_pass.cc b/impeller/entity/entity_pass.cc index 1e485b9c4781e..3c86f6eec3bfb 100644 --- a/impeller/entity/entity_pass.cc +++ b/impeller/entity/entity_pass.cc @@ -461,29 +461,11 @@ bool EntityPass::Render(ContentContext& renderer, // this method. auto color0 = root_render_target.GetColorAttachments().find(0u)->second; - // If a root stencil was provided by the caller, then verify that it has a - // configuration which can be used to render this pass. auto stencil_attachment = root_render_target.GetStencilAttachment(); auto depth_attachment = root_render_target.GetDepthAttachment(); - if (stencil_attachment.has_value() && depth_attachment.has_value()) { - auto stencil_texture = stencil_attachment->texture; - if (!stencil_texture) { - VALIDATION_LOG << "The root RenderTarget must have a stencil texture."; - return false; - } - - auto stencil_storage_mode = - stencil_texture->GetTextureDescriptor().storage_mode; - if (reads_from_onscreen_backdrop && - stencil_storage_mode == StorageMode::kDeviceTransient) { - VALIDATION_LOG << "The given root RenderTarget stencil needs to be read, " - "but it's marked as transient."; - return false; - } - } - // Setup a new root stencil with an optimal configuration if one wasn't - // provided by the caller. - else { + if (!stencil_attachment.has_value() || !depth_attachment.has_value()) { + // Setup a new root stencil with an optimal configuration if one wasn't + // provided by the caller. root_render_target.SetupDepthStencilAttachments( *renderer.GetContext(), *renderer.GetRenderTargetCache(), color0.texture->GetSize(),