[PATCH] Don't run diagnostic for sanitizer arguments several times

Alexey Samsonov samsonov at google.com
Thu Aug 8 01:08:04 PDT 2013


On Thu, Aug 8, 2013 at 3:48 AM, Richard Smith <richard at metafoo.co.uk> wrote:

>
>   Perhaps we should lazily construct and store a SanitizerArgs instance on
> the Driver, and share it between the various places that want to inspect
> the sanitizer arguments?
>

That's what first comes to mind. If it fits in the driver architecture and
doesn't add unwanted global state, I can implement this.

>
>   I don't think there's any good reason that the SanitizerArgs
> implementation mostly lives in the header. Since it's only included into
> two .cpp files, I don't think it matters much. Fix if you like :-)
>

Okay, will do.


>
> http://llvm-reviews.chandlerc.com/D1300
>



-- 
Alexey Samsonov, MSK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130808/41013889/attachment.html>


More information about the cfe-commits mailing list