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

[Impeller] revert non-zero tessellation optimization. #48234

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

jonahwilliams
Copy link
Member

@jonahwilliams jonahwilliams commented Nov 20, 2023

Forgot about the winding order. partially reverts #46282

See flutter/flutter#138598

Reopens flutter/flutter#135458

@jonahwilliams
Copy link
Member Author

Ping @dnfield

@matanlurey
Copy link
Contributor

What PR does this (partially) revert? Can you link in the PR description?

@jonahwilliams
Copy link
Member Author

Done, sorry thought I linked that.

Copy link
Contributor

@matanlurey matanlurey left a comment

Choose a reason for hiding this comment

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

RSLGTM based on it being a revert. Thanks!

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 27, 2023
@auto-submit auto-submit bot merged commit 3493f9b into flutter:main Nov 27, 2023
harryterkelsen pushed a commit to harryterkelsen/engine that referenced this pull request Nov 27, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 27, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 27, 2023
…139095)

flutter/engine@9fe1492...900ce09

2023-11-27 skia-flutter-autoroll@skia.org Roll Skia from 743fc1cde518 to 4c964f8c4738 (2 revisions) (flutter/engine#48410)
2023-11-27 jonahwilliams@google.com [Impeller] revert non-zero tessellation optimization. (flutter/engine#48234)
2023-11-27 skia-flutter-autoroll@skia.org Roll Skia from b0d81aba3f78 to 743fc1cde518 (3 revisions) (flutter/engine#48406)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
jonahwilliams added a commit to jonahwilliams/engine that referenced this pull request Nov 28, 2023
@jonahwilliams jonahwilliams deleted the revert_non_zero_wind branch November 28, 2023 16:46
caseycrogers pushed a commit to caseycrogers/flutter that referenced this pull request Dec 29, 2023
…lutter#139095)

flutter/engine@9fe1492...900ce09

2023-11-27 skia-flutter-autoroll@skia.org Roll Skia from 743fc1cde518 to 4c964f8c4738 (2 revisions) (flutter/engine#48410)
2023-11-27 jonahwilliams@google.com [Impeller] revert non-zero tessellation optimization. (flutter/engine#48234)
2023-11-27 skia-flutter-autoroll@skia.org Roll Skia from b0d81aba3f78 to 743fc1cde518 (3 revisions) (flutter/engine#48406)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
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.

2 participants