Skip to content
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][Compute] Guard subgroups with feature detection #40159

Merged
merged 3 commits into from
Mar 9, 2023

Conversation

dnfield
Copy link
Contributor

@dnfield dnfield commented Mar 9, 2023

This hasn't been a problem because the tests are always running on hardware that supports subgroups. This should give a better failure if you try to run the test on hardware that does not support subgroups, and will be more important as we get to runtime selection of which rendering strategies to use.

Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnfield dnfield added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 9, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 9, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Mar 9, 2023

auto label is removed for flutter/engine, pr: 40159, due to - The status or check suite Linux Fuchsia FEMU has failed. Please fix the issues identified (or deflake) before re-applying this label.

@dnfield dnfield added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 9, 2023
@auto-submit auto-submit bot merged commit 78ff6a0 into flutter:main Mar 9, 2023
@dnfield dnfield deleted the strokes branch March 9, 2023 04:00
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Mar 9, 2023
…lutter/engine#40159) (#122265)

Roll Flutter Engine from 7e1b3a98d114 to 78ff6a093411 (1 revision)
sourcegraph-bot pushed a commit to sgtest/megarepo that referenced this pull request Mar 9, 2023
…etection (flutter/engine#40159) (#122265)

Commit: 367ba011fddcb2e881b55039cdca16d9ec622140
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 9, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 10, 2023
hangyujin pushed a commit to hangyujin/flutter that referenced this pull request Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
None yet
2 participants