MTC added inline comments.

================
Comment at: test/Analysis/NewDelete-custom.cpp:7
 
-#if !(LEAKS && !ALLOCATOR_INLINING)
 // expected-no-diagnostics
 
----------------
Should we continue to keep this line?


================
Comment at: test/Analysis/NewDelete-sized-deallocation.cpp:1
+// RUN: %clang_analyze_cc1 -std=c++17 -analyzer-checker=core,cplusplus -verify 
-analyzer-output=text %s
+// RUN: %clang_analyze_cc1 -std=c++17 -analyzer-checker=core,cplusplus -verify 
-analyzer-output=text %s -fsized-deallocation
----------------
I believe `sized deallocation` is the feature since C++14, see 
https://en.cppreference.com/w/cpp/memory/new/operator_delete. 


https://reviews.llvm.org/D53543



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to