[libcxx-commits] [PATCH] D116477: Use `LLVM_COMMON_CMAKE_UTILS` in runtimes just for clarity

[John Ericson via Phabricator via libcxx-commits]

Ericson2314 marked 2 inline comments as done.
Ericson2314 added inline comments.


================
Comment at: compiler-rt/lib/builtins/CMakeLists.txt:13
+
+  set(COMPILER_RT_SOURCE_DIR}"${CMAKE_CURRENT_SOURCE_DIR}/../..")
+
----------------
mstorsjo wrote:
> Stray `}`, missing a space?
Oops! Sorry that got through


================
Comment at: libcxx/CMakeLists.txt:16
 # Add path for custom modules
-set(CMAKE_MODULE_PATH
+list(INSERT CMAKE_MODULE_PATH 0
   "${CMAKE_CURRENT_SOURCE_DIR}/cmake"
----------------
mstorsjo wrote:
> I presume this is fine (even if slightly unrelated) - I see that this aspect is inconsistent between runtimes today. It could be good to mention this in the commit message in any case.
Included now, with motivation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116477