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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Apr 6, 2022

Effectively reverts #31967, but keeps a flipped test.

See b/226484927

This created regressions for customer: money. It did not show any improvements I've been able to see in any benchmarks. Before enabling we need to have a motivating benchmark and should ideally verify that it does not regress internal benchmarks (though that is challenging right now).

@dnfield dnfield requested review from chinmaygarde and xster April 6, 2022 22:08
@flutter-dashboard flutter-dashboard bot added embedder Related to the embedder API platform-android labels Apr 6, 2022
@dnfield dnfield requested a review from zanderso April 6, 2022 22:08
@dnfield
Copy link
Contributor Author

dnfield commented Apr 6, 2022

@cbracken @jonahwilliams FYI since this is disabling stencil buffers on Windows, but again we had no real evidence of how or whether it was helping there.

@dnfield dnfield added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 6, 2022
@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Mac Unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

@fluttergithubbot fluttergithubbot removed the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

embedder Related to the embedder API platform-android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants