Hi Tim,

Yes - this is definitely overly conservative. I'll fix it up.

Thanks for catching this.

- Lang.

On Wed, Oct 24, 2012 at 4:28 AM, Tim Northover <[email protected]>wrote:

> Hi Lang,
>
> I think this check is ever-so-slightly too strict. Currently it
> disallows the following code:
>
> void foo(int a) {
>     #pragma ms_struct off
>     #pragma stdc fp_contract off
> }
>
> because the fp_contract pragma is no longer the first thing in the
> compound. However, the standard says it should "precede all explicit
> declarations and statements inside a compound statement". And since
> pragmas are neither of those, I think the construct should be allowed.
>
> Tim.
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to