Skip to content

Commit c239604

Browse files
author
Jeff Hammond
authored
[BuildBot] Reorder ARM and CUDA arg parsing (#2658)
ARM replaces x86 whereas CUDA appends, so the ARM part needs to come first for llvm_targets_to_build to be populated correctly. Signed-off-by: Hammond, Jeff R <[email protected]>
1 parent 31c23dd commit c239604

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

buildbot/configure.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,16 @@ def do_configure(args):
3232

3333
icd_loader_lib = os.path.join(icd_loader_lib, "libOpenCL.so" if platform.system() == 'Linux' else "OpenCL.lib")
3434

35+
# replace not append, so ARM ^ X86
36+
if args.arm:
37+
llvm_targets_to_build = 'ARM;AArch64'
38+
3539
if args.cuda:
3640
llvm_targets_to_build += ';NVPTX'
3741
llvm_enable_projects += ';libclc'
3842
libclc_targets_to_build = 'nvptx64--;nvptx64--nvidiacl'
3943
sycl_build_pi_cuda = 'ON'
4044

41-
# replace not append, so ARM ^ X86
42-
if args.arm:
43-
llvm_targets_to_build = 'ARM;AArch64'
44-
4545
if args.no_werror:
4646
sycl_werror = 'OFF'
4747

0 commit comments

Comments
 (0)