Hi Tim,<div><br></div><div>Yes - this is definitely overly conservative. I'll fix it up.</div><div><br></div><div>Thanks for catching this.</div><div><br></div><div>- Lang.<br><br><div class="gmail_quote">On Wed, Oct 24, 2012 at 4:28 AM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Hi Lang,<br>
<br>
I think this check is ever-so-slightly too strict. Currently it<br>
disallows the following code:<br>
<br>
void foo(int a) {<br>
    #pragma ms_struct off<br>
    #pragma stdc fp_contract off<br>
}<br>
<br>
because the fp_contract pragma is no longer the first thing in the<br>
compound. However, the standard says it should "precede all explicit<br>
declarations and statements inside a compound statement". And since<br>
pragmas are neither of those, I think the construct should be allowed.<br>
<br>
Tim.<br>
</div></div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>