[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