From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21445 invoked by alias); 11 Jan 2016 14:52:42 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 21434 invoked by uid 89); 11 Jan 2016 14:52:42 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=dfmode, DFmode, 6929, rounding X-HELO: mail-pa0-f45.google.com Received: from mail-pa0-f45.google.com (HELO mail-pa0-f45.google.com) (209.85.220.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Mon, 11 Jan 2016 14:52:41 +0000 Received: by mail-pa0-f45.google.com with SMTP id uo6so305508868pac.1 for ; Mon, 11 Jan 2016 06:52:40 -0800 (PST) X-Received: by 10.66.163.231 with SMTP id yl7mr176542068pab.141.1452523959021; Mon, 11 Jan 2016 06:52:39 -0800 (PST) Received: from gnu-tools-1.localdomain ([2602:304:cdc3:b260:208:caff:fef7:b10a]) by smtp.gmail.com with ESMTPSA id e82sm23955616pfb.76.2016.01.11.06.52.38 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 11 Jan 2016 06:52:38 -0800 (PST) Received: by gnu-tools-1.localdomain (Postfix, from userid 1000) id 6011A208E1; Mon, 11 Jan 2016 06:52:37 -0800 (PST) Date: Mon, 11 Jan 2016 14:52:00 -0000 From: "H.J. Lu" To: gcc-patches@gcc.gnu.org Cc: Uros Bizjak Subject: [PATCH] PR target/69225: gcc uses double precision instead of single float with -m32 -std=c99 -msoft-float Message-ID: <20160111145237.GA7821@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.24 (2015-08-30) X-IsSubscribed: yes X-SW-Source: 2016-01/txt/msg00597.txt.bz2 When FPU isn't used, there is no excess precision. We should set FLT_EVAL_METHOD to 0 if 387 is disabled. OK for trunk? Thanks. H.J. --- gcc/ PR target/69225 * config/i386/i386.h (TARGET_FLT_EVAL_METHOD): Set to 0 if TARGET_80387 is false. gcc/testsuite PR target/69225 * gcc.target/i386/pr69225-1.c: New test. * gcc.target/i386/pr69225-2.c: Likewise. * gcc.target/i386/pr69225-3.c: Likewise. * gcc.target/i386/pr69225-4.c: Likewise. * gcc.target/i386/pr69225-5.c: Likewise. * gcc.target/i386/pr69225-6.c: Likewise. --- gcc/config/i386/i386.h | 4 ++-- gcc/testsuite/gcc.target/i386/pr69225-1.c | 8 ++++++++ gcc/testsuite/gcc.target/i386/pr69225-2.c | 8 ++++++++ gcc/testsuite/gcc.target/i386/pr69225-3.c | 8 ++++++++ gcc/testsuite/gcc.target/i386/pr69225-4.c | 8 ++++++++ gcc/testsuite/gcc.target/i386/pr69225-5.c | 8 ++++++++ gcc/testsuite/gcc.target/i386/pr69225-6.c | 8 ++++++++ 7 files changed, 50 insertions(+), 2 deletions(-) create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-1.c create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-2.c create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-3.c create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-4.c create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-5.c create mode 100644 gcc/testsuite/gcc.target/i386/pr69225-6.c diff --git a/gcc/config/i386/i386.h b/gcc/config/i386/i386.h index dcaa011..5c97fd1 100644 --- a/gcc/config/i386/i386.h +++ b/gcc/config/i386/i386.h @@ -692,9 +692,9 @@ extern const char *host_detect_local_cpu (int argc, const char **argv); FPU, assume that the fpcw is set to extended precision; when using only SSE, rounding is correct; when using both SSE and the FPU, the rounding precision is indeterminate, since either may be chosen - apparently at random. */ + apparently at random. Also set to 0 if FPU isn't used. */ #define TARGET_FLT_EVAL_METHOD \ - (TARGET_MIX_SSE_I387 ? -1 : TARGET_SSE_MATH ? 0 : 2) + (TARGET_MIX_SSE_I387 ? -1 : (TARGET_SSE_MATH || !TARGET_80387) ? 0 : 2) /* Whether to allow x87 floating-point arithmetic on MODE (one of SFmode, DFmode and XFmode) in the current excess precision diff --git a/gcc/testsuite/gcc.target/i386/pr69225-1.c b/gcc/testsuite/gcc.target/i386/pr69225-1.c new file mode 100644 index 0000000..cc5b782 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-1.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-msse2 -mfpmath=sse" } */ + +#include + +#if FLT_EVAL_METHOD != 0 +# error FLT_EVAL_METHOD != 0 +#endif diff --git a/gcc/testsuite/gcc.target/i386/pr69225-2.c b/gcc/testsuite/gcc.target/i386/pr69225-2.c new file mode 100644 index 0000000..cd0a626 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-2.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-mno-sse -mno-80387" } */ + +#include + +#if FLT_EVAL_METHOD != 0 +# error FLT_EVAL_METHOD != 0 +#endif diff --git a/gcc/testsuite/gcc.target/i386/pr69225-3.c b/gcc/testsuite/gcc.target/i386/pr69225-3.c new file mode 100644 index 0000000..1440b74 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-3.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-m80387 -mfpmath=387" } */ + +#include + +#if FLT_EVAL_METHOD != 2 +# error FLT_EVAL_METHOD != 2 +#endif diff --git a/gcc/testsuite/gcc.target/i386/pr69225-4.c b/gcc/testsuite/gcc.target/i386/pr69225-4.c new file mode 100644 index 0000000..b032a5f --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-4.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-msse2 -mfancy-math-387 -mfpmath=sse" } */ + +#include + +#if FLT_EVAL_METHOD != 0 +# error FLT_EVAL_METHOD != 0 +#endif diff --git a/gcc/testsuite/gcc.target/i386/pr69225-5.c b/gcc/testsuite/gcc.target/i386/pr69225-5.c new file mode 100644 index 0000000..4162f77 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-5.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-msse2 -m80387 -mfpmath=sse,387" } */ + +#include + +#if FLT_EVAL_METHOD != -1 +# error FLT_EVAL_METHOD != -1 +#endif diff --git a/gcc/testsuite/gcc.target/i386/pr69225-6.c b/gcc/testsuite/gcc.target/i386/pr69225-6.c new file mode 100644 index 0000000..cd5f419 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr69225-6.c @@ -0,0 +1,8 @@ +/* { dg-do compile} */ +/* { dg-options "-mno-80387" } */ + +#include + +#if FLT_EVAL_METHOD != 0 +# error FLT_EVAL_METHOD != 0 +#endif -- 2.5.0