Skip to content

Conversation

@jason-simmons
Copy link
Member

See #177660

@github-actions github-actions bot added the engine flutter/engine related. See also e: labels. label Oct 28, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request disables Link-Time Optimization (LTO) for various Linux CI builder configurations by adding the --no-lto flag to the gn arguments. The changes are applied consistently across multiple JSON configuration files, including linux_arm_host_engine.json, linux_host_desktop_engine.json, linux_host_engine.json, linux_host_engine_ddm.json, linux_host_engine_test.json, and linux_unopt.json. In cases where the --no-lto flag was already present, it has been moved to maintain a consistent argument order. The changes are correct and align with the goal of disabling LTO for these Linux targets.

@zanderso zanderso requested a review from jtmcdole October 29, 2025 15:25
@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 29, 2025
@auto-submit auto-submit bot added this pull request to the merge queue Oct 29, 2025
Merged via the queue into flutter:master with commit df55910 Oct 29, 2025
186 of 187 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 29, 2025
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Oct 30, 2025
flutter/flutter@df72035...6f8abdd

2025-10-30 engine-flutter-autoroll@skia.org Roll Skia from 5035cdc7de31 to 18457971c30f (1 revision) (flutter/flutter#177767)
2025-10-30 engine-flutter-autoroll@skia.org Roll Skia from 018e2cdba2fe to 5035cdc7de31 (3 revisions) (flutter/flutter#177764)
2025-10-30 engine-flutter-autoroll@skia.org Roll Dart SDK from a0480f399f8f to 4785d5971d64 (21 revisions) (flutter/flutter#177760)
2025-10-30 engine-flutter-autoroll@skia.org Roll Skia from c803f12d2e26 to 018e2cdba2fe (1 revision) (flutter/flutter#177759)
2025-10-30 engine-flutter-autoroll@skia.org Roll Skia from 51267d4a2cea to c803f12d2e26 (2 revisions) (flutter/flutter#177756)
2025-10-30 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 3EF6k6lqXPWDwrdyj... to ksXeDDo2yYBXJ4uEu... (flutter/flutter#177754)
2025-10-30 30870216+gaaclarke@users.noreply.github.com impeller: allow setting image sampler uniforms by name (flutter/flutter#176749)
2025-10-30 engine-flutter-autoroll@skia.org Roll Skia from 0a0c9f8c704f to 51267d4a2cea (21 revisions) (flutter/flutter#177752)
2025-10-30 34871572+gmackall@users.noreply.github.com Copy symlinks when creating android cipd package, and update to package w/ symlinks (flutter/flutter#177638)
2025-10-30 1961493+harryterkelsen@users.noreply.github.com [web] Add GEMINI.md for web engine customizations (flutter/flutter#177413)
2025-10-30 99770169+lewinpauli@users.noreply.github.com Added computeDryBaseline implementation in RenderAligningShiftedBox (flutter/flutter#171250)
2025-10-29 36861262+QuncCccccc@users.noreply.github.com Refactor OverlayPortal semantics (flutter/flutter#173005)
2025-10-29 mdebbar@google.com [web] Delete unused canvaskit utils (flutter/flutter#177684)
2025-10-29 30870216+gaaclarke@users.noreply.github.com Fixed image links in //README.md (flutter/flutter#177750)
2025-10-29 jason-simmons@users.noreply.github.com Disable LTO in CI builder configurations for Linux targets (flutter/flutter#177694)
2025-10-29 mdebbar@google.com [web] Move webparagraph tests to their right location (flutter/flutter#177739)

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

To file a bug in Packages: 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 join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants