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 native decal on metal and Vulkan backend #40723
Conversation
.setAddressModeU(address_mode_u) | ||
.setAddressModeV(address_mode_v) | ||
.setAddressModeW(address_mode_w) | ||
.setBorderColor(vk::BorderColor::eFloatTransparentBlack) |
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.
I suspect that setting the border color if clamp to border isn't set should have no impact but I have not verified this yet.
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.
I can't run vulkan right now :(
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.
Verified that this works on Vulkan without extensions.
.setAddressModeU(address_mode_u) | ||
.setAddressModeV(address_mode_v) | ||
.setAddressModeW(address_mode_w) | ||
.setBorderColor(vk::BorderColor::eFloatTransparentBlack) |
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.
Verified that this works on Vulkan without extensions.
Fixes flutter/flutter#123619
Implements the decal tile mode with a clamp to border w/ transparent black border color. Should perform better by removing extra ops in the fragment shaders. This could be measurable in the blur case.