On Thu, Aug 1, 2013 at 12:46 PM, Hans Wennborg <h...@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.
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to