From patchwork Tue Jun 22 21:13:36 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aaron Sawdey X-Patchwork-Id: 1495858 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org (client-ip=2620:52:3:1:0:246e:9693:128c; helo=sourceware.org; envelope-from=gcc-patches-bounces+incoming=patchwork.ozlabs.org@gcc.gnu.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.a=rsa-sha256 header.s=default header.b=XhBomKwJ; dkim-atps=neutral Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4G8fKF2HJqz9sVm for ; Wed, 23 Jun 2021 07:14:07 +1000 (AEST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id D0F6E395540F for ; Tue, 22 Jun 2021 21:14:04 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D0F6E395540F DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1624396444; bh=Y52mNSwDnyFllEfewTag2OE+9KT8cei+AH2flVtJahE=; h=To:Subject:Date:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:Cc:From; b=XhBomKwJRnYeef7BTJhzejJwuSSPaVt9fRmEVp3eyVjIg6W+hznB9asi89cULoYUo ASNlU7OhkghRFTD1hNqu6gNWnZc4+AXYGe75+lvQfT3jK5lVGInLEAhuhYTFZHg6/z shEEyQyvP3ikCM5DAhjtUOgJ6CpzR+/tGnSjyPgI= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from ragesh4.localdomain (24-220-95-200-dynamic.midco.net [24.220.95.200]) by sourceware.org (Postfix) with ESMTP id 2C79C385742B for ; Tue, 22 Jun 2021 21:13:40 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 2C79C385742B Received: by ragesh4.localdomain (Postfix, from userid 501) id 3AA4E803F66E; Tue, 22 Jun 2021 16:13:38 -0500 (CDT) To: gcc-patches@gcc.gnu.org Subject: [PATCH, rs6000] Do not enable pcrel-opt by default Date: Tue, 22 Jun 2021 16:13:36 -0500 Message-Id: <20210622211336.29271-1-acsawdey@linux.ibm.com> X-Mailer: git-send-email 2.30.1 (Apple Git-130) MIME-Version: 1.0 X-Spam-Status: No, score=-10.3 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, KHOP_HELO_FCRDNS, RCVD_IN_PBL, RCVD_IN_SORBS_DUL, RDNS_DYNAMIC, SPF_HELO_NONE, SPF_SOFTFAIL, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Aaron Sawdey via Gcc-patches From: Aaron Sawdey Reply-To: Aaron Sawdey Cc: wschmidt@linux.ibm.com, segher@kernel.crashing.org Errors-To: gcc-patches-bounces+incoming=patchwork.ozlabs.org@gcc.gnu.org Sender: "Gcc-patches" SPEC2017 testing on p10 shows that this optimization does not have a positive impact on performance. So we are no longer going to enable it by default. The test cases for it needed to be updated so they always enable it to test it. OK for trunk and backport to 11 if bootstrap/regtest passes? Thanks! Aaron gcc/ * config/rs6000/rs6000-cpus.def: Take OPTION_MASK_PCREL_OPT out of OTHER_POWER10_MASKS so it will not be enabled by default. gcc/testsuite/ * gcc.target/powerpc/pcrel-opt-inc-di.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-df.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-di.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-hi.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-qi.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-sf.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-si.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-ld-vector.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-df.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-di.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-hi.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-qi.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-sf.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-si.c: Enable -mpcrel-opt to test it. * gcc.target/powerpc/pcrel-opt-st-vector.c: Enable -mpcrel-opt to test it. --- gcc/config/rs6000/rs6000-cpus.def | 3 ++- gcc/testsuite/gcc.target/powerpc/pcrel-opt-inc-di.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-df.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-di.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-hi.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-qi.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-sf.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-si.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-vector.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-df.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-di.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-hi.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-qi.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-sf.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-si.c | 2 +- gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-vector.c | 2 +- 16 files changed, 17 insertions(+), 16 deletions(-) diff --git a/gcc/config/rs6000/rs6000-cpus.def b/gcc/config/rs6000/rs6000-cpus.def index 52ce84835f7..1e8c9a68c3f 100644 --- a/gcc/config/rs6000/rs6000-cpus.def +++ b/gcc/config/rs6000/rs6000-cpus.def @@ -75,9 +75,10 @@ | OPTION_MASK_P9_VECTOR) /* Flags that need to be turned off if -mno-power10. */ +/* PCREL_OPT is now disabled by default so we comment it out here. */ #define OTHER_POWER10_MASKS (OPTION_MASK_MMA \ | OPTION_MASK_PCREL \ - | OPTION_MASK_PCREL_OPT \ + /* | OPTION_MASK_PCREL_OPT */ \ | OPTION_MASK_PREFIXED) #define ISA_3_1_MASKS_SERVER (ISA_3_0_MASKS_SERVER \ diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-inc-di.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-inc-di.c index c82041c9dc6..6272f5c72c3 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-inc-di.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-inc-di.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE unsigned int diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-df.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-df.c index d35862fcb6e..0dcab311add 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-df.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-df.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE double #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-di.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-di.c index 7e1ff99f20e..95b60f3b151 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-di.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-di.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE long long #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-hi.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-hi.c index 4143aeb7371..4a62dfbdf6e 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-hi.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-hi.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE unsigned short #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-qi.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-qi.c index 30d3236f95c..3a7aad45b55 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-qi.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-qi.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE unsigned char #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-sf.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-sf.c index 9d1e2a1956f..cb76bed04b8 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-sf.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-sf.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE float #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-si.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-si.c index 17be6fa1778..ad011d61448 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-si.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-si.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE int #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-vector.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-vector.c index 8c12aea5acd..c8f70b5d433 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-vector.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-ld-vector.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE vector double #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-df.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-df.c index d795d35d8de..686a3761462 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-df.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-df.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE double #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-di.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-di.c index bf57de4b886..fefe6d30e01 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-di.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-di.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE long long #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-hi.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-hi.c index 8822e767dfe..d2856863da7 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-hi.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-hi.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE unsigned short #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-qi.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-qi.c index 2f756833717..f617ffc6adf 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-qi.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-qi.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE unsigned char #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-sf.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-sf.c index 3dd88aad856..6eb20a3e047 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-sf.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-sf.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE float #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-si.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-si.c index 78dc8120efe..0cc0b304dd0 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-si.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-si.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE int #define LARGE 0x20000 diff --git a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-vector.c b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-vector.c index 2c602eb3103..d760819cfd2 100644 --- a/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-vector.c +++ b/gcc/testsuite/gcc.target/powerpc/pcrel-opt-st-vector.c @@ -1,6 +1,6 @@ /* { dg-do compile } */ /* { dg-require-effective-target powerpc_pcrel } */ -/* { dg-options "-O2 -mdejagnu-cpu=power10" } */ +/* { dg-options "-O2 -mdejagnu-cpu=power10 -mpcrel-opt" } */ #define TYPE vector double #define LARGE 0x20000