Re: [cfe-commits] [PATCH] Multiple pragmas at the start of a compound statement.

2012-11-03 Thread Lang Hames
Thanks John, Committed, with support for other pragma annotation tokens as well, in r167363. Cheers, Lang. On Thu, Nov 1, 2012 at 12:51 PM, John McCall rjmcc...@apple.com wrote: On Oct 25, 2012, at 1:39 PM, Lang Hames wrote: Tim Northover pointed out that there's a flaw in my recent

Re: [cfe-commits] [PATCH] Multiple pragmas at the start of a compound statement.

2012-11-01 Thread Lang Hames
Ping. On Thu, Oct 25, 2012 at 1:39 PM, Lang Hames lha...@gmail.com wrote: Hi All, Tim Northover pointed out that there's a flaw in my recent FP_CONTRACT patch (r166383). That patch restricted FP_CONTRACT (or rather tok::annot_pragma_fp_contract) to appearing as the first token in a

Re: [cfe-commits] [PATCH] Multiple pragmas at the start of a compound statement.

2012-11-01 Thread John McCall
On Oct 25, 2012, at 1:39 PM, Lang Hames wrote: Tim Northover pointed out that there's a flaw in my recent FP_CONTRACT patch (r166383). That patch restricted FP_CONTRACT (or rather tok::annot_pragma_fp_contract) to appearing as the first token in a compound statement. The problem with my

[cfe-commits] [PATCH] Multiple pragmas at the start of a compound statement.

2012-10-25 Thread Lang Hames
Hi All, Tim Northover pointed out that there's a flaw in my recent FP_CONTRACT patch (r166383). That patch restricted FP_CONTRACT (or rather tok::annot_pragma_fp_contract) to appearing as the first token in a compound statement. The problem with my patch is that it prohibits things like: void