tstellar added inline comments.
================
Comment at: clang/CMakeLists.txt:21-23
+ set(INCLUDE_DIRS "${LLVM_BINARY_DIR}/include" "${MAIN_INCLUDE_DIR}")
+ # Pull values from LLVMConfig.cmake.
+ set(MAIN_INCLUDE_DIR "${LLVM_INCLUDE_DIR}")
----------------
tstellar wrote:
> This doesn't look right (even though it passes my testing). I need to update
> this part of the patch.
I fixed this by removing of of the existing code that copied values from
LLVMConfig.cmake.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128777/new/
https://reviews.llvm.org/D128777
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
