On Aug 23, 2013, at 15:48 , Rafael Espíndola <[email protected]> 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.) Jordan _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
