r187602 - Options.td: O0 and O4 are not Joined options

Chandler Carruth chandlerc at google.com
Thu Aug 1 13:13:15 PDT 2013


On Thu, Aug 1, 2013 at 12:46 PM, Hans Wennborg <hans at hanshq.net> wrote:

> Author: hans
> Date: Thu Aug  1 14:46:33 2013
> New Revision: 187602
>
> URL: http://llvm.org/viewvc/llvm-project?rev=187602&view=rev
> Log:
> Options.td: O0 and O4 are not Joined options
>
> (I'm not sure how to test this because the Driver will still accept
>  e.g. "-O4foo", but it goes in the catch-all -O option instead of -O4.)
>

Maybe this is a sign that these shouldn't be here at all?

We should remove all handling of -O4 -- see my email from some time back
about rationalizing the -O options.

We should just have a joined -O flag, and translate -O0 appropriately IMO.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130801/77714047/attachment.html>


More information about the cfe-commits mailing list