Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove macOS impeller benchmarks #118917

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

Merged
merged 2 commits into from
Jan 21, 2023
Merged

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Jan 20, 2023

Impeller isn't enabled on macOS, so passing in enableImpeller: true isn't doing anything. Remove the benchmarks to avoid wasting capacity, since there are already non-impeller versions of these two benchmarks.

@jmagman jmagman added c: contributor-productivity Team-specific productivity, code health, technical debt. platform-mac Building on or for macOS specifically labels Jan 20, 2023
@jmagman jmagman requested a review from keyonghan as a code owner January 20, 2023 23:42
@jmagman jmagman self-assigned this Jan 20, 2023
@flutter-dashboard flutter-dashboard bot added the f: scrolling Viewports, list views, slivers, etc. label Jan 20, 2023
@jmagman jmagman requested a review from zanderso January 20, 2023 23:42
@jmagman jmagman added autosubmit Merge PR when tree becomes green via auto submit App and removed autosubmit Merge PR when tree becomes green via auto submit App labels Jan 20, 2023
@jmagman jmagman added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 20, 2023
@auto-submit auto-submit bot merged commit 25843bd into flutter:master Jan 21, 2023
@jmagman jmagman deleted the impeller-macos branch January 21, 2023 01:54
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Jan 21, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jan 21, 2023
* ec6ff90 Roll Flutter Engine from ccccee513fb2 to d84b3dc74c9f (2 revisions) (flutter/flutter#118893)

* 492d572 Cleanup obsolete --compact-async compiler option (flutter/flutter#118894)

* f291eb3 Remove unnecessary null checks in integration_test (flutter/flutter#118861)

* ab3c822 Remove unnecessary null checks in dev/devicelab (flutter/flutter#118842)

* bf72f5e 58eb1061e Revert "Remove references to Observatory (#38919)" (flutter/engine#39035) (flutter/flutter#118899)

* a07e8a6 [reland] Support wireless debugging (flutter/flutter#118895)

* 3c769ef Cupertino navbar ellipsis fix (flutter/flutter#118841)

* d1be731 3fead63ba Roll Dart SDK from ac4c63168ff2 to 03d35455a8d8 (1 revision) (flutter/engine#39036) (flutter/flutter#118909)

* c0ad6ad Marks Mac plugin_test_macos to be unflaky (flutter/flutter#118706)

* 8372001 Remove unnecessary null checks in flutter_test (flutter/flutter#118865)

* 288a773 Remove unnecessary null checks in flutter_driver (flutter/flutter#118864)

* bb73121 Remove unnecessary null checks in flutter/test (flutter/flutter#118905)

* 15bc4e4 Marks Mac_android microbenchmarks to be flaky (flutter/flutter#118693)

* 1cdaf9e e2c2e5009 [impeller] correct input order in ColorFilterContents::MakeBlend (flutter/engine#39038) (flutter/flutter#118913)

* 49e025d Update android defines test to use emulator (flutter/flutter#118808)

* bae4c1d Revert "Update android defines test to use emulator (#118808)" (flutter/flutter#118928)

* 9837eb6 Remove unnecessary null checks in flutter/rendering (flutter/flutter#118923)

* 25843bd Remove macOS impeller benchmarks (flutter/flutter#118917)

* 70cecf6 Remove unnecessary null checks in dev/*_tests (flutter/flutter#118844)

* c757df3 Remove unnecessary null checks in dev/bots (flutter/flutter#118846)

* 5d74b5c Remove unnecessary null checks in flutter/painting (flutter/flutter#118925)

* 7272c80 Remove unnecessary null checks in `flutter/{animation,semantics,scheduler}` (flutter/flutter#118922)

* 2baea2f 7b68d71b8 Roll Dart SDK from 03d35455a8d8 to 807077cc5d1b (1 revision) (flutter/engine#39042) (flutter/flutter#118933)

* 8d60a8c Roll Flutter Engine from 7b68d71b8d03 to 3a444b36657c (3 revisions) (flutter/flutter#118938)

* 5ccdb81 bb4e8dfe2 Roll Fuchsia Linux SDK from rPo4_TYHCtkoOfRup... to S6wQW1tLFe-YnReaZ... (flutter/engine#39048) (flutter/flutter#118942)

* b1f4070 ef7b1856a Roll Dart SDK from 8c2eb20b5376 to 548678dd684c (1 revision) (flutter/engine#39049) (flutter/flutter#118944)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Jan 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Jan 22, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Jan 22, 2023
auto-submit bot pushed a commit to flutter/plugins that referenced this pull request Jan 22, 2023
* 3c769ef Cupertino navbar ellipsis fix (flutter/flutter#118841)

* d1be731 3fead63ba Roll Dart SDK from ac4c63168ff2 to 03d35455a8d8 (1 revision) (flutter/engine#39036) (flutter/flutter#118909)

* c0ad6ad Marks Mac plugin_test_macos to be unflaky (flutter/flutter#118706)

* 8372001 Remove unnecessary null checks in flutter_test (flutter/flutter#118865)

* 288a773 Remove unnecessary null checks in flutter_driver (flutter/flutter#118864)

* bb73121 Remove unnecessary null checks in flutter/test (flutter/flutter#118905)

* 15bc4e4 Marks Mac_android microbenchmarks to be flaky (flutter/flutter#118693)

* 1cdaf9e e2c2e5009 [impeller] correct input order in ColorFilterContents::MakeBlend (flutter/engine#39038) (flutter/flutter#118913)

* 49e025d Update android defines test to use emulator (flutter/flutter#118808)

* bae4c1d Revert "Update android defines test to use emulator (#118808)" (flutter/flutter#118928)

* 9837eb6 Remove unnecessary null checks in flutter/rendering (flutter/flutter#118923)

* 25843bd Remove macOS impeller benchmarks (flutter/flutter#118917)

* 70cecf6 Remove unnecessary null checks in dev/*_tests (flutter/flutter#118844)

* c757df3 Remove unnecessary null checks in dev/bots (flutter/flutter#118846)

* 5d74b5c Remove unnecessary null checks in flutter/painting (flutter/flutter#118925)

* 7272c80 Remove unnecessary null checks in `flutter/{animation,semantics,scheduler}` (flutter/flutter#118922)

* 2baea2f 7b68d71b8 Roll Dart SDK from 03d35455a8d8 to 807077cc5d1b (1 revision) (flutter/engine#39042) (flutter/flutter#118933)

* 8d60a8c Roll Flutter Engine from 7b68d71b8d03 to 3a444b36657c (3 revisions) (flutter/flutter#118938)

* 5ccdb81 bb4e8dfe2 Roll Fuchsia Linux SDK from rPo4_TYHCtkoOfRup... to S6wQW1tLFe-YnReaZ... (flutter/engine#39048) (flutter/flutter#118942)

* b1f4070 ef7b1856a Roll Dart SDK from 8c2eb20b5376 to 548678dd684c (1 revision) (flutter/engine#39049) (flutter/flutter#118944)

* 8065887 Add transform flip (flutter/flutter#116705)

* 68b6e72 406dce64f Roll Fuchsia Mac SDK from ZTKDeVL1HDAwsZdhl... to l7jVM3Urw73TVWfee... (flutter/engine#39050) (flutter/flutter#118964)

* cf628ad aa194347a Roll Fuchsia Linux SDK from S6wQW1tLFe-YnReaZ... to l3c_b-vRr-o6ZFX_M... (flutter/engine#39055) (flutter/flutter#118968)

* f33e8d3 2a2dfaafb Roll Fuchsia Mac SDK from l7jVM3Urw73TVWfee... to 5TQ9IL4-Yu3KHCR-H... (flutter/engine#39056) (flutter/flutter#118969)
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
* 3c769ef Cupertino navbar ellipsis fix (flutter/flutter#118841)

* d1be731 3fead63ba Roll Dart SDK from ac4c63168ff2 to 03d35455a8d8 (1 revision) (flutter/engine#39036) (flutter/flutter#118909)

* c0ad6ad Marks Mac plugin_test_macos to be unflaky (flutter/flutter#118706)

* 8372001 Remove unnecessary null checks in flutter_test (flutter/flutter#118865)

* 288a773 Remove unnecessary null checks in flutter_driver (flutter/flutter#118864)

* bb73121 Remove unnecessary null checks in flutter/test (flutter/flutter#118905)

* 15bc4e4 Marks Mac_android microbenchmarks to be flaky (flutter/flutter#118693)

* 1cdaf9e e2c2e5009 [impeller] correct input order in ColorFilterContents::MakeBlend (flutter/engine#39038) (flutter/flutter#118913)

* 49e025d Update android defines test to use emulator (flutter/flutter#118808)

* bae4c1d Revert "Update android defines test to use emulator (#118808)" (flutter/flutter#118928)

* 9837eb6 Remove unnecessary null checks in flutter/rendering (flutter/flutter#118923)

* 25843bd Remove macOS impeller benchmarks (flutter/flutter#118917)

* 70cecf6 Remove unnecessary null checks in dev/*_tests (flutter/flutter#118844)

* c757df3 Remove unnecessary null checks in dev/bots (flutter/flutter#118846)

* 5d74b5c Remove unnecessary null checks in flutter/painting (flutter/flutter#118925)

* 7272c80 Remove unnecessary null checks in `flutter/{animation,semantics,scheduler}` (flutter/flutter#118922)

* 2baea2f 7b68d71b8 Roll Dart SDK from 03d35455a8d8 to 807077cc5d1b (1 revision) (flutter/engine#39042) (flutter/flutter#118933)

* 8d60a8c Roll Flutter Engine from 7b68d71b8d03 to 3a444b36657c (3 revisions) (flutter/flutter#118938)

* 5ccdb81 bb4e8dfe2 Roll Fuchsia Linux SDK from rPo4_TYHCtkoOfRup... to S6wQW1tLFe-YnReaZ... (flutter/engine#39048) (flutter/flutter#118942)

* b1f4070 ef7b1856a Roll Dart SDK from 8c2eb20b5376 to 548678dd684c (1 revision) (flutter/engine#39049) (flutter/flutter#118944)

* 8065887 Add transform flip (flutter/flutter#116705)

* 68b6e72 406dce64f Roll Fuchsia Mac SDK from ZTKDeVL1HDAwsZdhl... to l7jVM3Urw73TVWfee... (flutter/engine#39050) (flutter/flutter#118964)

* cf628ad aa194347a Roll Fuchsia Linux SDK from S6wQW1tLFe-YnReaZ... to l3c_b-vRr-o6ZFX_M... (flutter/engine#39055) (flutter/flutter#118968)

* f33e8d3 2a2dfaafb Roll Fuchsia Mac SDK from l7jVM3Urw73TVWfee... to 5TQ9IL4-Yu3KHCR-H... (flutter/engine#39056) (flutter/flutter#118969)
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 c: contributor-productivity Team-specific productivity, code health, technical debt. f: scrolling Viewports, list views, slivers, etc. platform-mac Building on or for macOS specifically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants