[PATCH] D37860: [MSan] Enable use-after-dtor instrumentation by default.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 16:08:57 PDT 2017


morehouse added a comment.

In https://reviews.llvm.org/D37860#871368, @eugenis wrote:

> Looking at __sanitizer_dtor_callback implementation, this change will add a (fast) stack unwind in every destructor. In extreme cases (like a tight loop doing string operations) it could be bad for performance. I've seen ~15% AFAIR.


I haven't looked at performance, but I'll take a look at your internal bug.  Until that is resolved, this can be on hold.


https://reviews.llvm.org/D37860





More information about the llvm-commits mailing list