sebastian-ne added inline comments.

================
Comment at: llvm/CMakeLists.txt:209
   "${CMAKE_CURRENT_SOURCE_DIR}/cmake/modules"
+  "${LLVM_COMMON_CMAKE_UTILS}/Modules"
   )
----------------
Hi, adding this module path overwrites the `llvm_check_linker_flag` from 
`llvm/cmake/modules/LLVMCheckLinkerFlag.cmake` with the one defined in 
`cmake/Modules/CheckLinkerFlag.cmake`, which does not check the linker but the 
compiler.
This breaks our gcc+ld build because ld does not support `--color-diagnostics`.

Our downstream issue is here: 
https://github.com/GPUOpen-Drivers/llpc/issues/1645


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116521

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D116521: [CMake... Sebastian Neubauer via Phabricator via cfe-commits

Reply via email to