On Wed, May 4, 2011 at 4:22 PM, Rainer Orth <r...@cebitec.uni-bielefeld.de> wrote: > Hi Michael, > >> -Ofast is intended to be -O3 plus -ffast-math. For the compiler proper >> this works, but under -ffast-math we add crtfastmath.o (or some >> equivalent) to the link command line on some targets. As usual for our >> specs this uses matching on command line arguments, hence we'll >> explicitely have to add Ofast. >> >> All these targets use the same idiom, so I hope it's okay if I'm testing >> only x86_64-linux (regstrapping in progress there). CCed the respective >> target maintainers. I wouldn't have anything against a global reviewer >> just acking the whole thing :) > > I had noticed that duplication before. Rather than continue down that > road, I'd rather have us introduce a new spec for this issue.
This can be done separately. Note the patch also applies to the 4.6 branch where -Ofast was introduced. Thus, ok for trunk and the branch. Thanks, Richard.
