[PATCH] Pragma optimize on/off

Sean Silva chisophugis at gmail.com
Mon May 12 11:36:00 PDT 2014


+#pragma clang optimize on top of spaghetti  // expected-warning {{extra
tokens at end of '#pragma clang optimize' - ignored}}

The diagnostic here is ambiguous. Was the pragma itself ignored or were the
extra tokens ignored?

-- Sean Silva

PS: I actually laughed out loud when I read "on top of spaghetti" :)


On Thu, May 8, 2014 at 10:43 AM, Dario Domizioli
<dario.domizioli at gmail.com>wrote:

> Hello again!
>
> I have now extended the patch with support for serialization of the state
> of the pragma (for PCHs). I have followed a similar pattern to the one used
> for the floating point pragmas and the diagnostic pragmas.
> I have added a PCH test that verifies that if a "#pragma clang optimize
> off" is still active at the end of a PCH then a source file compiled
> including the PCH will behave as if the pragma was specified in the source
> (as it happens with normal headers). The test is modeled after the pragma
> diagnostics test.
>
> I hope the new attached patch covers the issue that was raised by
> Richard... but I welcome any feedback. I might have missed something.
>
> Cheers,
>     Dario Domizioli
>     SN Systems - Sony Computer Entertainment Group
>
>
>
>
>
>
>
>
> On 7 May 2014 18:41, Dario Domizioli <dario.domizioli at gmail.com> wrote:
>
>>
>> Thanks Aaron!
>>
>> Patch LGTM, modulo Richard's comments about serialization (I worried
>>> about the same thing, and my preference is for the patch to be
>>> all-inclusive when feasible, but Richard is welcome to weigh in). I
>>> have no strong opinions on whether it should be one diagnostic or two.
>>>
>>
>> I will wait for Richard's comments then; meanwhile I'll keep working on
>> the serialization issue.
>>
>> As for the diagnostics, I think that having two might make it easier to
>> extend the "unexpected" case if the pragma gains new functionality in the
>> future, while the "missing" case is unlikely to change... although the
>> %select is quite powerful so I haven't got a strong preference either.
>>
>> Cheers,
>>     Dario Domizioli
>>     SN Systems - Sony Computer Entertainment Group
>>
>>
>>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140512/fd930f49/attachment.html>


More information about the cfe-commits mailing list