From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25297 invoked by alias); 29 Apr 2004 17:34:07 -0000 Mailing-List: contact libc-hacker-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-hacker-owner@sources.redhat.com Received: (qmail 25280 invoked from network); 29 Apr 2004 17:34:07 -0000 Received: from unknown (HELO sunsite.ms.mff.cuni.cz) (195.113.15.26) by sources.redhat.com with SMTP; 29 Apr 2004 17:34:07 -0000 Received: from sunsite.ms.mff.cuni.cz (sunsite.mff.cuni.cz [127.0.0.1]) by sunsite.ms.mff.cuni.cz (8.12.8/8.12.8) with ESMTP id i3TFM03j027231; Thu, 29 Apr 2004 17:22:00 +0200 Received: (from jakub@localhost) by sunsite.ms.mff.cuni.cz (8.12.8/8.12.8/Submit) id i3TFM0FI027227; Thu, 29 Apr 2004 17:22:00 +0200 Date: Thu, 29 Apr 2004 17:34:00 -0000 From: Jakub Jelinek To: Ulrich Drepper , "M.H.VanLeeuwen" Cc: Glibc hackers Subject: [PATCH] Fix [BZ #139] (followup) Message-ID: <20040429152159.GU5191@sunsite.ms.mff.cuni.cz> Reply-To: Jakub Jelinek References: <20040429052647.GQ5191@sunsite.ms.mff.cuni.cz> <4090FF37.11B1A8A1@megsinet.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4090FF37.11B1A8A1@megsinet.net> User-Agent: Mutt/1.4i X-SW-Source: 2004-04/txt/msg00095.txt.bz2 On Thu, Apr 29, 2004 at 08:12:23AM -0500, M.H.VanLeeuwen wrote: > Jakub, > > Could you also check qp_qtoui.c ... > What about qp_qtoux.c ... > And qp_qtox.c Obviously you're right. This patch is needed in addition to the previous one. 2004-04-29 Jakub Jelinek * sysdeps/sparc/sparc64/soft-fp/qp_qtoui.c (_Qp_qtoui): Use %f31 for single precision register, add it to __asm clobbers. * sysdeps/sparc/sparc64/soft-fp/qp_qtoux.c (_Qp_qtoux): Use fqtox instead of fqtoi in QP_HANDLE_EXCEPTIONS. * sysdeps/sparc/sparc64/soft-fp/qp_qtox.c (_Qp_qtox): Likewise. Reported by M. H. VanLeeuwen . --- libc/sysdeps/sparc/sparc64/soft-fp/qp_qtoui.c.jj 2001-07-06 06:56:05.000000000 +0200 +++ libc/sysdeps/sparc/sparc64/soft-fp/qp_qtoui.c 2004-04-29 15:15:11.942798201 +0200 @@ -1,6 +1,6 @@ /* Software floating-point emulation. Return (unsigned int)(*a) - Copyright (C) 1997,1999 Free Software Foundation, Inc. + Copyright (C) 1997, 1999, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Richard Henderson (rth@cygnus.com) and Jakub Jelinek (jj@ultra.linux.cz). @@ -38,9 +38,9 @@ unsigned int _Qp_qtoui(const long double __asm ( " ldd [%1], %%f52\n" " ldd [%1+8], %%f54\n" -" fqtoi %%f52, %%f60\n" -" st %%f60, [%0]\n" -" " : : "r" (&rx), "r" (a) : QP_CLOBBER); +" fqtoi %%f52, %%f31\n" +" st %%f31, [%0]\n" +" " : : "r" (&rx), "r" (a) : QP_CLOBBER, "f31"); r = rx); return r; --- libc/sysdeps/sparc/sparc64/soft-fp/qp_qtoux.c.jj 2001-07-06 06:56:05.000000000 +0200 +++ libc/sysdeps/sparc/sparc64/soft-fp/qp_qtoux.c 2004-04-29 15:16:37.753419820 +0200 @@ -1,6 +1,6 @@ /* Software floating-point emulation. Return (unsigned long)(*a) - Copyright (C) 1997,1999 Free Software Foundation, Inc. + Copyright (C) 1997, 1999, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Richard Henderson (rth@cygnus.com) and Jakub Jelinek (jj@ultra.linux.cz). @@ -38,7 +38,7 @@ unsigned long _Qp_qtoux(const long doubl __asm ( " ldd [%1], %%f52\n" " ldd [%1+8], %%f54\n" -" fqtoi %%f52, %%f60\n" +" fqtox %%f52, %%f60\n" " std %%f60, [%0]\n" " " : : "r" (&rx), "r" (a) : QP_CLOBBER); r = rx); --- libc/sysdeps/sparc/sparc64/soft-fp/qp_qtox.c.jj 2001-07-06 06:56:05.000000000 +0200 +++ libc/sysdeps/sparc/sparc64/soft-fp/qp_qtox.c 2004-04-29 15:16:57.549872038 +0200 @@ -1,6 +1,6 @@ /* Software floating-point emulation. Return (long)(*a) - Copyright (C) 1997,1999 Free Software Foundation, Inc. + Copyright (C) 1997, 1999, 2004 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Richard Henderson (rth@cygnus.com) and Jakub Jelinek (jj@ultra.linux.cz). @@ -38,7 +38,7 @@ long _Qp_qtox(const long double *a) __asm ( " ldd [%1], %%f52\n" " ldd [%1+8], %%f54\n" -" fqtoi %%f52, %%f60\n" +" fqtox %%f52, %%f60\n" " std %%f60, [%0]\n" " " : : "r" (&rx), "r" (a) : QP_CLOBBER); r = rx); Jakub