Fix NDEBUG build: guard call to dump with #if/#endif

kparzysz · kparzysz · commit 5a77bdc3e7d6 · 2024-03-14T11:47:50.000-05:00
diff --git a/flang/lib/Lower/OpenMP/Clauses.cpp b/flang/lib/Lower/OpenMP/Clauses.cpp
@@ -101,7 +101,9 @@ struct SymbolAndDesignatorExtractor {
     } else {
       // This could still be a Substring or ComplexPart, but at least Substring
       // is not allowed in OpenMP.
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
       maybeDsg->dump();
+#endif
       llvm_unreachable("Expecting DataRef designator");
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,9 @@ struct SymbolAndDesignatorExtractor {`
`101`	`101`	`} else {`
`102`	`102`	`// This could still be a Substring or ComplexPart, but at least Substring`
`103`	`103`	`// is not allowed in OpenMP.`
	`104`	`+#if !defined(NDEBUG) \|\| defined(LLVM_ENABLE_DUMP)`
`104`	`105`	`maybeDsg->dump();`
	`106`	`+#endif`
`105`	`107`	`llvm_unreachable("Expecting DataRef designator");`
`106`	`108`	`}`
`107`	`109`	`}`