New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Impeller] Use blit for root pass copy #39933
Conversation
9491bb5
to
1d5c261
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RSLGTM
1d5c261
to
b826efc
Compare
@@ -514,7 +514,7 @@ TEST_P(RendererTest, CanBlitTextureToTexture) { | |||
// Blit `bridge` to the top left corner of the texture. | |||
pass->AddCopy(bridge, texture); | |||
|
|||
pass->EncodeCommands(context->GetResourceAllocator()); | |||
assert(pass->EncodeCommands(context->GetResourceAllocator())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assert
s get stripped away in !defined(NDEBUG). Besides, let's not terminate the process on this error. Return false like elsewhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, fixed.
Really supportive of this change except for the stripped away thing that you want to do :) |
b826efc
to
437eea6
Compare
Use a blit for the root pass copy when available in the backend.