[llvm-commits] [llvm-gcc-4.2] r82079 - /llvm-gcc-4.2/trunk/gcc/opts.c

[Evan Cheng]

On Sep 16, 2009, at 11:52 PM, Török Edwin wrote:

> On 2009-09-17 03:36, Evan Cheng wrote:
>> On Sep 16, 2009, at 5:02 PM, Eli Friedman wrote:
>>
>>
>>> On Wed, Sep 16, 2009 at 2:02 PM, Evan Cheng <evan.cheng at apple.com>
>>> wrote:
>>>
>>>> Author: evancheng
>>>> Date: Wed Sep 16 16:02:55 2009
>>>> New Revision: 82079
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=82079&view=rev
>>>> Log:
>>>> -O now means -O2. rdar://7228174.
>>>>
>>> I think this change needs further explanation... why exactly do we
>>> want llvm-gcc to be inconsistent with gcc?
>>>
>>
>> It's Chris' decision. See the "[LLVMdev] FYI: Phoronix GCC vs. LLVM-
>> GCC benchmarks" thread.
>>
>
> Chris said there that -O1 should be equivalent to -Os, not -O2.
> So which one is it?

This patch doesn't change -O1. It changes -O to mean -O2, not -O1.

Evan

>
> Best regards,
> --Edwin