[PATCH] Add a no_sanitize_vptr function attribute.
Oliver Chang
ochang at google.com
Fri Apr 17 10:25:08 PDT 2015
On 17 April 2015 at 06:31, Aaron Ballman <aaron at aaronballman.com> wrote:
>
> I agree with Richard's comments about this being extended to apply to
> a list of sanitizers. I wish we would have done that for the
> no_sanitize_blah attributes.
>
> Also, why no C++-style attribute?
>
You mean something like GCC<"no_sanitize_vtpr"> instead?
> I think some more documentation would be nice. People aren't always
> familiar with what those vptr checks would look like, or why they
> might want to have them removed. What's the benefit to applying this
> attribute, how would you decide when it's appropriate to add it, etc.
>
Would that documentation belong here?
>
>
> Why is the shell required for this test?
>
I forgot to delete this when I copied the file from another test. Removed.
--
Cheers,
Oliver
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150417/740252d2/attachment.html>
More information about the cfe-commits
mailing list