Re: PATCH: PR target/15383: __float128 function name is used for long double

["H. J. Lu" <hjl at lucon dot org>]

On Wed, May 19, 2004 at 03:12:26PM -0700, Jim Wilson wrote:
> On Wed, 2004-05-19 at 13:00, H. J. Lu wrote:
> > soft-fp in glibc doesn't have them. Are they used by gcc in TFmode on
> > ia64? How do they work for HPUX?
> 
> I didn't check.  See
> gcc/testsuite/gcc.c-torture/execute/ieee/fp-cmp-[48].c which use builtin
> functions that required unordered checks.  These only test double, but
> we should also have similar float and long double tests, and for IA-64,
> the other long double (128-bit for linux, 80-bit for HPUX).  There may
> also be some Fortran testcases, I didn't look.  It looks like the
> builtin functions take any type, so just changing the types in the
> testcase should work.  Just using __float128, without your patch, I get
> calls to __unordtf2, along with the others like __eqtf2.
> 

I will add a few more testcases and try to implement unordered functions
in soft-fp.

> For HPUX, it appears that the support is already there.  The HPUX
> _U_Qfcmp function has a bit for unordered and ia64_expand_compare has
> code to use it for unordered compares.
> 
> Support for UNEQ etc is not there, but we can ignore those for now.  All
> we need is for UNORD to work.
> 
> > Here is the updated patch.
> 
> Thanks.  This looks OK.
> 
> I note that the ia64_sysv4_init_libfuncs function doesn't have an
> explanatory comment in front of it, but that is a nit.  Just add one
> before you check in the patch please.

My previous patch is for 3.4. Is that OK for 3.4 with comment change?

Here is one for mainline. But mainline is broken for all 64bit
platforms now:

http://gcc.gnu.org/ml/gcc-patches/2004-05/msg01182.html



H.J.

Attachment: gcc-ia64-float128-1.patch
Description: Text document

Attachment: gcc-3.4-ia64-float128-5.patch
Description: Text document