[PATCH] D41408: [analyzer] NFC: Fix nothrow operator new definition in a test.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 20 11:23:37 PST 2017


NoQ added inline comments.


================
Comment at: test/Analysis/NewDelete-custom.cpp:7
 
-#if !LEAKS
+#if !(LEAKS && !ALLOCATOR_INLINING)
 // expected-no-diagnostics
----------------
a.sidorin wrote:
> Double negation can be simplified a bit: `#if !LEAKS || ALLOCATOR_INLINING`
The rest of the `#if`s in this file look like `#if LEAKS && !ALLOCATOR_INLINING`, so i wanted this to look similarly.


Repository:
  rC Clang

https://reviews.llvm.org/D41408





More information about the cfe-commits mailing list