ARM PATCH: Fix fix patterns

Richard Henderson rth@redhat.com
Mon Mar 10 19:22:00 GMT 2003


On Mon, Mar 10, 2003 at 11:35:15AM +0000, Richard Earnshaw wrote:
> Richard, In reply to your comment in that PR, the documentation for FIX is 
> very confusingly written.  There are two entries for FIX in rtl.texi, one 
> for fix:MODE_FLOAT and one for fix:MODE_INT.  The first describes a 
> rounding operation and the second a conversion operation.  Maybe the 
> former should be given a different RTL code (ROUND?).

Well, really we need all of the round operations: crop to
zero, round to nearest, floor, ceil, round a-la the setting
in the fp control register.

We want all of this so that we can implement the full set
of c99 rounding functions as builtins.

As for (fix:INT (reg:FLOAT)) vs (fix:INT (fix:FLOAT (reg:FLOAT)))...
Well, sometimes I don't see the point of the argument.  We could
as well change the documentation such that this *isn't* undefined
for non-integral quantities, and save the creation of extra rtl.



r~



More information about the Gcc-patches mailing list