[PATCH] D53543: [analyzer] MallocChecker: pr39348: Realize that sized delete isn't a custom delete.

[Henry Wong via Phabricator via cfe-commits]

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