thakis added inline comments.

================
Comment at: clang/CMakeLists.txt:311
+  "SystemZ Default Arch")
+add_definitions( -DCLANG_SYSTEMZ_DEFAULT_ARCH="${CLANG_SYSTEMZ_DEFAULT_ARCH}")
+
----------------
Hahnfeld wrote:
> Passing values like this is unusual for CMake and causes all source files to 
> be recompiled if the value is changed. Instead could we add this to 
> `include/clang/Config/config.h.cmake` like other `CLANG_DEFAULT_*` options?
+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75914



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

Reply via email to