On 05/30/2011 12:07 PM, Alexandre Oliva wrote: > One of my patches for PR48866 mysteriously regressed fma3-fma.c. It > turned out that a rearrangement in the order of emitting insns during > expansion caused combine to attempt combinations in a different order, > and this ended up exposing a latent error in the cost computation of FMA > insns, that caused a desirable combination to be regarded as not > profitable. The problem was that, when stripping out NEGs from > operands, that would be costless in FMA variants, we took the > sub-operand from the wrong variable. Oops. > > I took the opportunity to fix the names of the fma_fnm* patterns, that > were missing the “n”. > > Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install?
Ok. Bernd
