[PATCH] D141907: [CMake] Ensure `CLANG_RESOURCE_DIR` is respected

Junchang Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 4 17:01:58 PDT 2023


paperchalice added inline comments.


================
Comment at: cmake/Modules/GetClangResourceDir.cmake:13
+  if(DEFINED CLANG_RESOURCE_DIR AND NOT CLANG_RESOURCE_DIR STREQUAL "")
+    set(ret_dir bin/${CLANG_RESOURCE_DIR})
+  else()
----------------
tstellar wrote:
> Why is the bin prefix here?
See https://clang.llvm.org/doxygen/classclang_1_1driver_1_1Driver.html#acda8dfdf4f80efa84df98157e1779152
`GetResourcesPath` will return `<prefix>/lib/<version>` when `CLANG_RESOURCE_DIR` is empty, `<prefix>/bin/CLANG_RESOURCE_DIR` otherwise.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141907



More information about the cfe-commits mailing list