[cfe-commits] r166383 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.td lib/Parse/ParseStmt.cpp test/CodeGen/fp-contract-pragma.cpp test/Parser/pragma-fp-contract.c
Tim Northover
t.p.northover at gmail.com
Wed Oct 24 04:28:30 PDT 2012
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.
More information about the cfe-commits
mailing list