[PATCH] D97855: [runtimes] Fix crosscompiling after a7cad6680b4087eff8994f1f99ac40c661a6621f (D97451)

Wed Mar 3 08:10:01 PST 2021

zero9178 added a comment.

Sorry for the oversight and thanks for taking a look!

================
Comment at: llvm/cmake/modules/LLVMExternalProjectUtils.cmake:242
+    set(compiler_args -DCMAKE_ASM_COMPILER=${CMAKE_ASM_COMPILER}
+                      -DCMAKE_ASM_COMPILER_TARGET=${ARG_TARGET_TRIPLE}
+                      -DCMAKE_C_COMPILER=${CMAKE_C_COMPILER}
----------------
I think this could potentially cause issues since callers might not specify TARGET_TRIPLE. Would it maybe make sense to move the block from line 214 further down to also affect the cross compiling build? 

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97855/new/

https://reviews.llvm.org/D97855