<div dir="ltr"><div>LGTM</div><div><br></div>On Fri, Sep 13, 2013 at 11:26 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">================<br></div><div class="im">
Comment at: lib/Lex/Pragma.cpp:1106<br>
@@ +1105,3 @@<br>
+        // Collect the warning ids.<br>
+        SmallVector<int, 1> Ids;<br>
+        PP.Lex(Tok);<br>
----------------<br>
</div><div class="im">Richard Smith wrote:<br>
> Only 1 inline ID? Cheapskate! =) Maybe 10? 16?<br>
</div>These are usually issued one by one.  :)</blockquote><div><br></div><div>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.<br>
</div></div></div></div>