r189149 - Don't imply -flto with -O4.

Jordan Rose jordan_rose at apple.com
Fri Aug 23 16:02:17 PDT 2013

On Aug 23, 2013, at 15:48 , Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:

>>> We now saturate at -O3.
>> We actually don't...-O4 still implies -fvectorize. Otherwise I would say -O4 should warn that it doesn't do anything any more, at least for one release.
> We do the same for -O3:
> $ ./build/bin/clang -c -O4 -v test.c  > o4.log 2>&1
> $ ./build/bin/clang -c -O3 -v test.c  > o3.log 2>&1
> $ diff o3.log  o4.log
> I agree that warning at -OX with X > 3 is probably a good idea.

Well then. I guess shouldEnableVectorizerAtOLevel() could be simpler! (I did not read it all the way through, only noticed that it had a case for O4.)


More information about the cfe-commits mailing list