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

Use simarm_x64 when targeting arm #10010

Merged
merged 8 commits into from
Aug 1, 2019
Merged

Use simarm_x64 when targeting arm #10010

merged 8 commits into from
Aug 1, 2019

Conversation

liamappelbe
Copy link
Contributor

@liamappelbe liamappelbe commented Jul 22, 2019

32-bit binaries are being deprecated on Mac, so when building for arm we need to use Dart's simarm_x64 mode, which builds a 64-bit version of gen_snapshot that can produce 32-bit arm AOT snapshots.

This is a breaking change, because there's a small change needed in flutter/flutter to support this: flutter/flutter#36710

Testing: I built the flutter examples in this mode using Xcode 10 and verified that they work on an iPhone 4S.
Bug: flutter/flutter#22598

@liamappelbe liamappelbe changed the title Use simarm_x64 when targeting arm on Mac Use simarm_x64 when targeting arm Jul 22, 2019
@liamappelbe liamappelbe marked this pull request as ready for review July 26, 2019 17:34
@liamappelbe liamappelbe requested a review from cbracken July 29, 2019 16:56
Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

@liamappelbe liamappelbe merged commit 9fca3c7 into flutter:master Aug 1, 2019
liamappelbe added a commit to flutter/flutter that referenced this pull request Aug 1, 2019
* Remove multi-arch check in iOS builds

* Manual roll of flutter/engine#10010
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 2, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 2, 2019
[email protected]:flutter/engine.git/compare/bf9288597586...eac7ef0

git log bf92885..eac7ef0 --no-merges --oneline
2019-08-01 [email protected] Add copy_gen_snapshots.py tool (flutter/engine#10430)
2019-08-01 [email protected] Fix mac gen_snapshot uploader (flutter/engine#10423)
2019-08-01 [email protected] Make kernel compiler use host toolchain (flutter/engine#10419)
2019-08-01 [email protected] Use simarm_x64 when targeting arm (flutter/engine#10010)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
cfontas pushed a commit to cfontas/engine that referenced this pull request Aug 8, 2019
* GN changes for simarm_x64

* GN changes for simarm_x64

* Cleanup a bit

* Guard simarm_x64 behind a flag

* Remove flag

* Remove Mac specificity
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants