[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
 // 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.

  rC Clang


More information about the cfe-commits mailing list