[all-commits] [llvm/llvm-project] 3e2a6d: [llvm][cmake] Fix add_subdirectory build in multi-...

Sebastian Neubauer via All-commits all-commits at lists.llvm.org
Tue Dec 20 02:57:41 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3e2a6d7f0317706ad5de93b26b51fda00e493c91
      https://github.com/llvm/llvm-project/commit/3e2a6d7f0317706ad5de93b26b51fda00e493c91
  Author: Sebastian Neubauer <Sebastian.Neubauer at amd.com>
  Date:   2022-12-20 (Tue, 20 Dec 2022)

  Changed paths:
    M llvm/cmake/modules/AddLLVM.cmake
    M llvm/cmake/modules/CMakeLists.txt

  Log Message:
  -----------
  [llvm][cmake] Fix add_subdirectory build in multi-config

Using CMAKE_CFG_INTDIR in paths that are used in configure_file,
resulted in a folder that is literally called '${CONFIGURATION}'
for the multi-config ninja build.

I think this is a regression from a while ago. Fix this by replacing
CMAKE_CFG_INTDIR with '.'. We can only create one of the
LLVMConfig.cmake files as the consuming CMake project can only import a
single file. This creates LLVMConfig.cmake and others in the place where
they were previously and where they are for a single-config build.

Differential Revision: https://reviews.llvm.org/D139623




More information about the All-commits mailing list