rnk added inline comments.
================
Comment at: llvm/include/llvm/Config/llvm-config.h.cmake:92
/* Define to 1 if you have the <sysexits.h> header file. */
#cmakedefine HAVE_SYSEXITS_H ${HAVE_SYSEXITS_H}
----------------
This is unrelated, but appears to be in the wrong header, it should be in
config.h. It isn't namespaced, and could conflict with another project's config
header.
================
Comment at: llvm/include/llvm/Config/llvm-config.h.cmake:95
+/* Define to 1 to enable the experimental new pass manager by default */
+#cmakedefine01 ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER
+
----------------
This should be namespaced in LLVM_*.
Can we drop "experimental" now to shorten it? LLVM_ENABLE_NEW_PASS_MANAGER?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92072/new/
https://reviews.llvm.org/D92072
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
