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

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 20 03:42:05 PST 2017


a.sidorin accepted this revision.
a.sidorin added a comment.
This revision is now accepted and ready to land.

Looks good, just a minor nit.



================
Comment at: test/Analysis/NewDelete-custom.cpp:7
 
-#if !LEAKS
+#if !(LEAKS && !ALLOCATOR_INLINING)
 // expected-no-diagnostics
----------------
Double negation can be simplified a bit: `#if !LEAKS || ALLOCATOR_INLINING`


Repository:
  rC Clang

https://reviews.llvm.org/D41408





More information about the cfe-commits mailing list