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

Commit e8d49f1

Browse files
committed
Don't use intel clang on arm64 macs
1 parent 077b6f0 commit e8d49f1

File tree

4 files changed

+2
-27
lines changed

4 files changed

+2
-27
lines changed

ci/builders/mac_clang_tidy.json

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
"debug",
1616
"--prebuilt-dart-sdk",
1717
"--no-lto",
18-
"--force-mac-arm64",
1918
"--target-dir",
2019
"ci/host_debug_clang_tidy",
2120
"--rbe",
@@ -44,7 +43,6 @@
4443
"debug",
4544
"--simulator",
4645
"--no-lto",
47-
"--force-mac-arm64",
4846
"--target-dir",
4947
"ci/ios_debug_sim_clang_tidy",
5048
"--rbe",
@@ -87,7 +85,6 @@
8785
"debug",
8886
"--prebuilt-dart-sdk",
8987
"--no-lto",
90-
"--force-mac-arm64",
9188
"--target-dir",
9289
"ci/host_debug_clang_tidy",
9390
"--rbe",
@@ -137,7 +134,6 @@
137134
"debug",
138135
"--prebuilt-dart-sdk",
139136
"--no-lto",
140-
"--force-mac-arm64",
141137
"--target-dir",
142138
"ci/host_debug_clang_tidy",
143139
"--rbe",
@@ -187,7 +183,6 @@
187183
"debug",
188184
"--prebuilt-dart-sdk",
189185
"--no-lto",
190-
"--force-mac-arm64",
191186
"--target-dir",
192187
"ci/host_debug_clang_tidy",
193188
"--rbe",
@@ -237,7 +232,6 @@
237232
"debug",
238233
"--prebuilt-dart-sdk",
239234
"--no-lto",
240-
"--force-mac-arm64",
241235
"--target-dir",
242236
"ci/host_debug_clang_tidy",
243237
"--rbe",
@@ -288,7 +282,6 @@
288282
"debug",
289283
"--prebuilt-dart-sdk",
290284
"--no-lto",
291-
"--force-mac-arm64",
292285
"--target-dir",
293286
"ci/host_debug_clang_tidy",
294287
"--rbe",
@@ -306,7 +299,6 @@
306299
"debug",
307300
"--simulator",
308301
"--no-lto",
309-
"--force-mac-arm64",
310302
"--target-dir",
311303
"ci/ios_debug_sim_clang_tidy",
312304
"--rbe",

ci/builders/mac_ios_engine.json

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@
33
{
44
"drone_dimensions": [
55
"device_type=none",
6-
"os=Mac-13|Mac-14",
7-
"cpu=x86"
6+
"os=Mac-13|Mac-14"
87
],
98
"gclient_variables": {
109
"download_android_deps": false,
@@ -231,8 +230,7 @@
231230
{
232231
"drone_dimensions": [
233232
"device_type=none",
234-
"os=Mac-13|Mac-14",
235-
"cpu=x86"
233+
"os=Mac-13|Mac-14"
236234
],
237235
"gclient_variables": {
238236
"download_android_deps": false,

ci/builders/mac_unopt.json

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,6 @@
354354
"--unoptimized",
355355
"--no-lto",
356356
"--prebuilt-dart-sdk",
357-
"--force-mac-arm64",
358357
"--mac-cpu",
359358
"arm64",
360359
"--rbe",
@@ -415,7 +414,6 @@
415414
"debug",
416415
"--simulator",
417416
"--no-lto",
418-
"--force-mac-arm64",
419417
"--simulator-cpu",
420418
"arm64",
421419
"--rbe",
@@ -483,7 +481,6 @@
483481
"debug",
484482
"--simulator",
485483
"--no-lto",
486-
"--force-mac-arm64",
487484
"--simulator-cpu",
488485
"arm64",
489486
"--darwin-extension-safe",

tools/gn

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -502,11 +502,6 @@ def to_gn_args(args):
502502
gn_args['host_cpu'] = 'x86'
503503
gn_args['current_cpu'] = 'x86'
504504

505-
# TODO(cbracken):
506-
# https://github.com/flutter/flutter/issues/103386
507-
if gn_args['target_os'] == 'ios' and not args.force_mac_arm64:
508-
gn_args['host_cpu'] = 'x64'
509-
510505
if gn_args['target_os'] == 'ios' or get_host_os() == 'mac':
511506
if gn_args['target_os'] == 'ios':
512507
gn_args['use_ios_simulator'] = args.simulator
@@ -917,13 +912,6 @@ def parse_args(args):
917912
parser.add_argument('--ios', dest='target_os', action='store_const', const='ios')
918913
parser.add_argument('--mac', dest='target_os', action='store_const', const='mac')
919914
parser.add_argument('--mac-cpu', type=str, choices=['x64', 'arm64'], default='x64')
920-
parser.add_argument(
921-
'--force-mac-arm64',
922-
action='store_true',
923-
default=False,
924-
help='Force use of the clang_arm64 toolchain on an arm64 mac host when '
925-
'building host binaries.'
926-
)
927915
parser.add_argument('--simulator', action='store_true', default=False)
928916
parser.add_argument('--linux', dest='target_os', action='store_const', const='linux')
929917
parser.add_argument('--fuchsia', dest='target_os', action='store_const', const='fuchsia')

0 commit comments

Comments
 (0)