PATCH: PR target/15383: __float128 function name is used for long double
Zack Weinberg
zack@codesourcery.com
Fri May 14 02:03:00 GMT 2004
Jim Wilson <wilson@specifixinc.com> writes:
> I think this would be clearer if it was changed to check cmptf_libcall
> instead of TARGET_HPUX. That makes the connection between
> ia64_hpux_init_libfuncs a direct connection, instead of an indirect
> connection via TARGET_HPUX. It might also be useful to add an abort
> for the case where GET_MODE (op) == TFmode and cmptf_libcall is not
> set. In that case, we would fall through and generate an
> unrecognizable insn. It is perhaps better to abort, or give a suitable
> error, than to generate an unrecognizable insn. We should never reach
> here with TFmode if eq_optab etc are set.
Agree.
> Your set of comparisons by the way is incomplete. There is also
> unord_optab. This matters for FP comparisons like ordered, unordered,
> uneq, unge, ungt, unle, unlt, and ltgt.
Last I checked ia64.md didn't have the branch expanders for those -
hence the FUTURE: comment in ia64_expand_compare. Hmm, I see there is
now [bs]ordered and [bs]unordered but not the others.
zw
More information about the Gcc-patches
mailing list