[PATCH] D75914: systemz: allow configuring default CLANG_SYSTEMZ_ARCH
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 30 11:24:25 PDT 2020
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
More information about the cfe-commits
mailing list