Skip to content

Commit 86c0411

Browse files
committed
[lldb][sanitized] Don't overwrite TARGETS_TO_BUILD
We moved the sanitized bot to arm64 but the LLVM we were building was only X86-enabled. So when the freshly built Clang was used to compile the runtimes it failed on arm64. This was happening because we were overwriting the TARGETS_TO_BUILD for the sanitized variant. There's no need to overwrite it. We should just build for X86 *AND* AArch64, which would get us some more LLDB test coverage too.
1 parent 340f4c4 commit 86c0411

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

zorg/jenkins/build.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -568,7 +568,6 @@ def lldb_cmake_builder(target, variant=None):
568568
cmake_cmd.append('-DLLDB_ENABLE_LUA=On')
569569

570570
if variant == 'sanitized':
571-
cmake_cmd.append('-DLLVM_TARGETS_TO_BUILD=X86')
572571
cmake_cmd.append('-DLLVM_USE_SANITIZER=Address;Undefined')
573572
# There is no need to compile the lldb tests with an asanified compiler
574573
# if we have a host compiler available.

0 commit comments

Comments
 (0)