nikic added inline comments.
================
Comment at: llvm/cmake/modules/AddLLVM.cmake:890
+    configure_file(
+      ${LLVM_MAIN_SRC_DIR}/cmake/driver-template.cpp.in
+      ${CMAKE_CURRENT_BINARY_DIR}/${name}-driver.cpp)
----------------
chapuni wrote:
> Do you have a plan to export the template?
Unsurprisingly, this breaks standalone builds. However, I think that the 
multicall tool is fundamentally incompatible with standalone builds, in the 
sense that all tools included in it must be part of the initial build 
configuration -- you can't first build the LLVM multicall tool and then add 
clang to it afterwards.

So I think we just need to guard this code by LLVM_TOOL_LLVM_DRIVER_BUILD, so 
we don't try to read a non-exported driver-template.cpp.in file even if we're 
not actually building the multicall tool.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109977

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to