This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: PowerPC Floating Point compare bug?


>>>>> Geoff Keating writes:

Geoff> I think the code simply has the wrong expansion in the movsfcc
Geoff> define_expand for this (rare) case---the operands need to be
Geoff> interchanged.

	The problem is that one cannot tell when the operands should be
treated reversed without doing another max/min pass, so it just isn't
worth it.  I have removed the "NE" case from the patterns.

Geoff> That code is bad in other ways, too, it doesn't allow for overflow and
Geoff> NaNs and such.

	I agree and I thought that it was conditionalized on fast-math at
one point.  I would have expected that movMMcc would be conditionalized on
fast-math or when it knows the operations will be safe (as in this case
with constants and compiler-generated values).

David


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]