[PATCH] D91094: fix some Wundef warnings in public headers

Jameson Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 17:19:35 PST 2020


This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG8f1e0dcbb090: fix some Wundef warnings in public headers (authored by vtjnash).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91094

Files:
  llvm/include/llvm/ADT/Statistic.h
  llvm/include/llvm/Config/abi-breaking.h.cmake


Index: llvm/include/llvm/Config/abi-breaking.h.cmake
===================================================================
--- llvm/include/llvm/Config/abi-breaking.h.cmake
+++ llvm/include/llvm/Config/abi-breaking.h.cmake
@@ -20,7 +20,7 @@
 
 /* Allow selectively disabling link-time mismatch checking so that header-only
    ADT content from LLVM can be used without linking libSupport. */
-#if !LLVM_DISABLE_ABI_BREAKING_CHECKS_ENFORCING
+#if !defined(LLVM_DISABLE_ABI_BREAKING_CHECKS_ENFORCING) || !LLVM_DISABLE_ABI_BREAKING_CHECKS_ENFORCING
 
 // ABI_BREAKING_CHECKS protection: provides link-time failure when clients build
 // mismatch with LLVM
Index: llvm/include/llvm/ADT/Statistic.h
===================================================================
--- llvm/include/llvm/ADT/Statistic.h
+++ llvm/include/llvm/ADT/Statistic.h
@@ -36,6 +36,8 @@
 // configure time.
 #if !defined(NDEBUG) || LLVM_FORCE_ENABLE_STATS
 #define LLVM_ENABLE_STATS 1
+#else
+#define LLVM_ENABLE_STATS 0
 #endif
 
 namespace llvm {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91094.307221.patch
Type: text/x-patch
Size: 1016 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201124/0dcd7f66/attachment.bin>


More information about the llvm-commits mailing list