[PATCH] Lex and ignore Microsoft's #pragma warning(...)
Richard Smith
richard at metafoo.co.uk
Fri Sep 13 13:28:22 PDT 2013
LGTM
On Fri, Sep 13, 2013 at 11:26 AM, Reid Kleckner <rnk at google.com> wrote:
> ================
> Comment at: lib/Lex/Pragma.cpp:1106
> @@ +1105,3 @@
> + // Collect the warning ids.
> + SmallVector<int, 1> Ids;
> + PP.Lex(Tok);
> ----------------
> Richard Smith wrote:
> > Only 1 inline ID? Cheapskate! =) Maybe 10? 16?
> These are usually issued one by one. :)
IIRC I've written code which had about 5 or 6 warning numbers listed in a
single pragma. But whatever you prefer, this is never going to matter in
practice.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130913/c625b065/attachment.html>
More information about the cfe-commits
mailing list