[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