[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