Skip to content

Commit 26bdff8

Browse files
authored
Merge pull request #80253 from edymtt/edymtt/use-runtimes-build-quater-enabled-by-default
Use LLVM_USE_RUNTIMES to build compiler-rt by default
2 parents 2746a83 + c983cac commit 26bdff8

File tree

4 files changed

+4
-8
lines changed

4 files changed

+4
-8
lines changed

utils/build_swift/build_swift/driver_arguments.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1404,7 +1404,7 @@ def create_argument_parser():
14041404
'separated options "-DCMAKE_VAR1=YES,-DCMAKE_VAR2=/tmp". Can '
14051405
'be called multiple times to add multiple such options.')
14061406

1407-
option('--llvm-build-compiler-rt-with-use-runtimes', toggle_true,
1407+
option('--llvm-build-compiler-rt-with-use-runtimes', toggle_true, default=True,
14081408
help='Switch to LLVM_ENABLE_RUNTIMES as the mechanism to build compiler-rt'
14091409
'It will become the default with LLVM 21, this flag is '
14101410
'meant to stage its introduction and account for edge cases')

utils/build_swift/tests/expected_options.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@
231231
'lldb_build_variant': 'Debug',
232232
'lldb_build_with_xcode': '0',
233233
'llvm_assertions': True,
234-
'llvm_build_compiler_rt_with_use_runtimes': False,
234+
'llvm_build_compiler_rt_with_use_runtimes': True,
235235
'llvm_build_variant': 'Debug',
236236
'llvm_cmake_options': [],
237237
'llvm_enable_modules': False,

validation-test/BuildSystem/llvm-build-compiler-rt-with-enable-external-compiler-rt.test

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
# REQUIRES: standalone_build
22

3-
# RUN: %empty-directory(%t)
4-
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --cmake %cmake 2>&1 | %FileCheck --check-prefix=EXTERNAL-COMPILER-RT-CHECK %s
53
# RUN: %empty-directory(%t)
64
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes=0 --cmake %cmake 2>&1 | %FileCheck --check-prefix=EXTERNAL-COMPILER-RT-CHECK %s
75

@@ -12,8 +10,6 @@
1210
# EXTERNAL-COMPILER-RT-CHECK-SAME: -DLLVM_ENABLE_PROJECTS{{[^ ]*}}={{[^ ]*}}compiler-rt
1311
# EXTERNAL-COMPILER-RT-CHECK-SAME: llvm
1412

15-
# RUN: %empty-directory(%t)
16-
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --skip-build-compiler-rt --cmake %cmake 2>&1 | %FileCheck --check-prefix=DONT-BUILD-COMPILER-RT %s
1713
# RUN: %empty-directory(%t)
1814
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes=0 --skip-build-compiler-rt --cmake %cmake 2>&1 | %FileCheck --check-prefix=DONT-BUILD-COMPILER-RT %s
1915

validation-test/BuildSystem/llvm-build-compiler-rt-with-use-runtimes.test

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# REQUIRES: standalone_build
22

33
# RUN: %empty-directory(%t)
4-
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes --cmake %cmake 2>&1 | %FileCheck --check-prefix=LLVM-USE-RUNTIMES %s
4+
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --cmake %cmake 2>&1 | %FileCheck --check-prefix=LLVM-USE-RUNTIMES %s
55
# RUN: %empty-directory(%t)
66
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes=1 --cmake %cmake 2>&1 | %FileCheck --check-prefix=LLVM-USE-RUNTIMES %s
77

@@ -14,7 +14,7 @@
1414
# LLVM-USE-RUNTIMES-SAME: llvm
1515

1616
# RUN: %empty-directory(%t)
17-
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes --skip-build-compiler-rt --cmake %cmake 2>&1 | %FileCheck --check-prefix=DONT-BUILD-COMPILER-RT %s
17+
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --skip-build-compiler-rt --cmake %cmake 2>&1 | %FileCheck --check-prefix=DONT-BUILD-COMPILER-RT %s
1818
# RUN: %empty-directory(%t)
1919
# RUN: SKIP_XCODE_VERSION_CHECK=1 SWIFT_BUILD_ROOT=%t %swift_src_root/utils/build-script --dry-run --skip-build --llvm-build-compiler-rt-with-use-runtimes=1 --skip-build-compiler-rt --cmake %cmake 2>&1 | %FileCheck --check-prefix=DONT-BUILD-COMPILER-RT %s
2020

0 commit comments

Comments
 (0)