Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Impeller] Allow toggling vulkan validation using a command line test flag. #40728

Merged
merged 3 commits into from
Mar 29, 2023

Conversation

chinmaygarde
Copy link
Member

The --enable_vulkan_validation playground flag controls this instead of needing to patch sources.

… flag.

The `--enable_vulkan_validation` playground flag controls this instead
of needing to patch sources.
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

@chinmaygarde chinmaygarde self-assigned this Mar 28, 2023
@chinmaygarde chinmaygarde added e: impeller autosubmit Merge PR when tree becomes green via auto submit App labels Mar 28, 2023
Copy link
Member

@bdero bdero left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -13,6 +13,7 @@ PlaygroundSwitches::PlaygroundSwitches(const fml::CommandLine& args) {
if (args.GetOptionValue("playground_timeout_ms", &timeout_str)) {
timeout = std::chrono::milliseconds(atoi(timeout_str.c_str()));
}
enable_vulkan_validation = args.HasOption("enable_vulkan_validation");
Copy link
Member

Choose a reason for hiding this comment

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

Nit: It's possible to test this flag flip in a reasonable way via switches_unittests.cc these days

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

auto-submit bot commented Mar 28, 2023

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

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

auto-submit bot commented Mar 28, 2023

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

@chinmaygarde chinmaygarde added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 28, 2023
@auto-submit auto-submit bot merged commit 6941c18 into flutter:main Mar 29, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 29, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 29, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 29, 2023
zanderso pushed a commit to flutter/flutter that referenced this pull request Mar 29, 2023
…123664)

* 1d506dfd7 [Impeller] Allow metal shaders to compile through SPIR-V with openGL semantics. (flutter/engine#40616)

* 3287e2b36 [Impeller] support half precision uniforms and half precision samplers (flutter/engine#40590)

* 5c1c07fba Revert "Revert "Reland "Default the CanvasKit base URL to local artifacts. (#40293)" (#40470)" (#40700)" (flutter/engine#40717)

* 9e4cedc81 Forward stdout and stderr from dart2wasm when verbose. (flutter/engine#40731)

* 6941c1820 [Impeller] Allow toggling vulkan validation using a command line test flag. (flutter/engine#40728)

* 6ef595829 Roll buildroot to build CanvasKit for speed instead of code size (flutter/engine#40737)

* 5b8e02479 [Impeller] Gaussian blur: Add alpha mask specialization (flutter/engine#40707)

* 6852bea71 Roll Skia from 7311e9220faf to e3eeabb14e9c (4 revisions) (flutter/engine#40745)

* 235e42bea Add an option to malioc_diff.py to print a unified diff (flutter/engine#40732)

* 38e6d772f Roll Dart SDK from 1acd1e649fb7 to 69867ba60bb7 (2 revisions) (flutter/engine#40738)

* d81c4f2c7 [Impeller] migrate texture fill shaders to half precision. (flutter/engine#40735)
@chinmaygarde chinmaygarde deleted the valflag branch March 29, 2023 06:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants