[PATCH] D75970: Fix compiler warning when compiling without asserts

Abhay Raj Singh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 10 19:42:05 PDT 2020


rathod-sahaab created this revision.
rathod-sahaab added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

This patch aims to prevent warning-as-error failures in release build.
As suggested in this comment
https://reviews.llvm.org/D69930#1910922


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D75970

Files:
  llvm/lib/Transforms/IPO/OpenMPOpt.cpp


Index: llvm/lib/Transforms/IPO/OpenMPOpt.cpp
===================================================================
--- llvm/lib/Transforms/IPO/OpenMPOpt.cpp
+++ llvm/lib/Transforms/IPO/OpenMPOpt.cpp
@@ -44,7 +44,7 @@
 STATISTIC(NumOpenMPRuntimeFunctionUsesIdentified,
           "Number of OpenMP runtime function uses identified");
 
-#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
+#if !defined(NDEBUG)
 static constexpr auto TAG = "[" DEBUG_TYPE "]";
 #endif
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75970.249542.patch
Type: text/x-patch
Size: 465 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200311/f2d9fed4/attachment.bin>


More information about the llvm-commits mailing list