Skip to content

Commit b3b24eb

Browse files
committed
[build-script] Add plumbing for llbuild assertions
This adds two flags to the build script to enable/disable assertions in llbuild: --llbuild-assertions, --no-llbuild-assertions The default value is taken from the global assertions flag.
1 parent 0b9c973 commit b3b24eb

File tree

4 files changed

+17
-1
lines changed

4 files changed

+17
-1
lines changed

utils/build-script

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -488,6 +488,8 @@ class BuildScriptInvocation(object):
488488
args.swift_stdlib_assertions).lower(),
489489
"--swift-analyze-code-coverage", str(
490490
args.swift_analyze_code_coverage).lower(),
491+
"--llbuild-enable-assertions", str(
492+
args.llbuild_assertions).lower(),
491493
"--cmake-generator", args.cmake_generator,
492494
"--build-jobs", str(args.build_jobs),
493495
"--common-cmake-options=%s" % ' '.join(

utils/build-script-impl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2354,7 +2354,7 @@ for host in "${ALL_HOSTS[@]}"; do
23542354
-DLIT_EXECUTABLE:PATH="${LLVM_SOURCE_DIR}/utils/lit/lit.py"
23552355
-DFILECHECK_EXECUTABLE:PATH="$(build_directory_bin ${LOCAL_HOST} llvm)/FileCheck"
23562356
-DCMAKE_BUILD_TYPE:STRING="${LLBUILD_BUILD_TYPE}"
2357-
-DLLVM_ENABLE_ASSERTIONS:BOOL=$(true_false "${LLBUILD_ENABLE_ASSERTIONS}")
2357+
-DLLBUILD_ENABLE_ASSERTIONS:BOOL=$(true_false "${LLBUILD_ENABLE_ASSERTIONS}")
23582358
)
23592359
;;
23602360
swiftpm)

utils/build_swift/driver_arguments.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,9 @@ def _apply_default_arguments(args):
105105
if args.swift_stdlib_assertions is None:
106106
args.swift_stdlib_assertions = args.assertions
107107

108+
if args.llbuild_assertions is None:
109+
args.llbuild_assertions = args.assertions
110+
108111
# Set the default CMake generator.
109112
if args.cmake_generator is None:
110113
args.cmake_generator = 'Ninja'
@@ -642,6 +645,13 @@ def create_argument_parser():
642645
const=False,
643646
help='disable assertions in LLDB')
644647

648+
option('--llbuild-assertions', store,
649+
const=True,
650+
help='enable assertions in llbuild')
651+
option('--no-llbuild-assertions', store('llbuild_assertions'),
652+
const=False,
653+
help='disable assertions in llbuild')
654+
645655
# -------------------------------------------------------------------------
646656
in_group('Select the CMake generator')
647657

utils/build_swift/tests/expected_options.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,7 @@
135135
'libdispatch_build_variant': 'Debug',
136136
'libicu_build_variant': 'Debug',
137137
'lit_args': '-sv',
138+
'llbuild_assertions': True,
138139
'lldb_assertions': None,
139140
'lldb_build_variant': 'Debug',
140141
'lldb_build_with_xcode': '1',
@@ -356,6 +357,7 @@ class IgnoreOption(_BaseOption):
356357
SetOption('--cmark-assertions', value=True),
357358
SetOption('--lldb-assertions', value=True),
358359
SetOption('--llvm-assertions', value=True),
360+
SetOption('--llbuild-assertions', value=True),
359361
SetOption('--swift-assertions', value=True),
360362
SetOption('--swift-stdlib-assertions', value=True),
361363
SetOption('-T', dest='validation_test', value=True),
@@ -367,6 +369,8 @@ class IgnoreOption(_BaseOption):
367369
SetOption('--no-assertions', dest='assertions', value=False),
368370
SetOption('--no-lldb-assertions', dest='lldb_assertions', value=False),
369371
SetOption('--no-llvm-assertions', dest='llvm_assertions', value=False),
372+
SetOption('--no-llbuild-assertions',
373+
dest='llbuild_assertions', value=False),
370374
SetOption('--no-swift-assertions', dest='swift_assertions', value=False),
371375
SetOption('--no-swift-stdlib-assertions',
372376
dest='swift_stdlib_assertions', value=False),

0 commit comments

Comments
 (0)