[PATCH] D36757: Add C++17 aligned new/delete interceptors to standalone lsan

Francis Ricci via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 14:57:44 PDT 2017


fjricci added inline comments.


================
Comment at: lib/lsan/lsan_interceptors.cc:305
 { OPERATOR_DELETE_BODY; }
 
 #endif  // !SANITIZER_MAC
----------------
alekseyshl wrote:
> fjricci wrote:
> > alekseyshl wrote:
> > > Don't we need to intercept by mangled names here too?
> > ASan currently does not. Shouldn't be hard to add the mangled name interception for both ASan and LSan in a second patch (unless you'd rather have it all here).
> Why not, let's add them in this patch.
Sounds good.


https://reviews.llvm.org/D36757





More information about the llvm-commits mailing list