On 17 April 2015 at 06:31, Aaron Ballman <[email protected]> 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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to