Message ID | 20230106122552.145679-1-guojiufu@linux.ibm.com |
---|---|
State | New |
Headers |
Return-Path: <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 535A73858C53 for <patchwork@sourceware.org>; Fri, 6 Jan 2023 12:26:27 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 535A73858C53 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1673007987; bh=mIIekyfyQPOMB5oTbW6zHmiMd7RxVgpKKu2O7JQYWis=; h=To:Cc:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=M9OFCw4mcV2qm/EiWPesUyMv1y7jSfG5baKe/fiRqi3y0itzlkKWZ1GrvUjpHdfNQ BDyjkBRse2loZ3iMeaPryBNJDH/Oxg4QoSb/u7sTkvUsrVbiVf9j2Ko72Kufu9ioKy 3LTMyEzAVA7WvSuafAXvE2SzfczUcHNB+JwNyopQ= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by sourceware.org (Postfix) with ESMTPS id EDAF73858D28; Fri, 6 Jan 2023 12:25:59 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org EDAF73858D28 Received: from pps.filterd (m0098419.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3069XbrC008384; Fri, 6 Jan 2023 12:25:59 GMT Received: from pps.reinject (localhost [127.0.0.1]) by mx0b-001b2d01.pphosted.com (PPS) with ESMTPS id 3mxdyxpmev-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 06 Jan 2023 12:25:59 +0000 Received: from m0098419.ppops.net (m0098419.ppops.net [127.0.0.1]) by pps.reinject (8.17.1.5/8.17.1.5) with ESMTP id 306BiwFU016105; Fri, 6 Jan 2023 12:25:58 GMT Received: from ppma03ams.nl.ibm.com (62.31.33a9.ip4.static.sl-reverse.com [169.51.49.98]) by mx0b-001b2d01.pphosted.com (PPS) with ESMTPS id 3mxdyxpme7-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 06 Jan 2023 12:25:58 +0000 Received: from pps.filterd (ppma03ams.nl.ibm.com [127.0.0.1]) by ppma03ams.nl.ibm.com (8.17.1.19/8.17.1.19) with ESMTP id 3063KUA2007699; Fri, 6 Jan 2023 12:25:57 GMT Received: from smtprelay04.fra02v.mail.ibm.com ([9.218.2.228]) by ppma03ams.nl.ibm.com (PPS) with ESMTPS id 3mtcq6g8qa-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 06 Jan 2023 12:25:56 +0000 Received: from smtpav04.fra02v.mail.ibm.com (smtpav04.fra02v.mail.ibm.com [10.20.54.103]) by smtprelay04.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 306CPsTK20382126 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 6 Jan 2023 12:25:54 GMT Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 870112004B; Fri, 6 Jan 2023 12:25:54 +0000 (GMT) Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 74B1720040; Fri, 6 Jan 2023 12:25:53 +0000 (GMT) Received: from pike.rch.stglabs.ibm.com (unknown [9.5.12.127]) by smtpav04.fra02v.mail.ibm.com (Postfix) with ESMTP; Fri, 6 Jan 2023 12:25:53 +0000 (GMT) To: gcc-patches@gcc.gnu.org Cc: segher@kernel.crashing.org, dje.gcc@gmail.com, linkw@gcc.gnu.org, guojiufu@linux.ibm.com Subject: [PATCH] rs6000: mark tieable between INT and FLOAT Date: Fri, 6 Jan 2023 20:25:52 +0800 Message-Id: <20230106122552.145679-1-guojiufu@linux.ibm.com> X-Mailer: git-send-email 2.17.1 X-TM-AS-GCONF: 00 X-Proofpoint-ORIG-GUID: BTM5mG6geYugwb7GLva6ukHybZ85eD7n X-Proofpoint-GUID: mKzltAbtTkTt4in31z127iyflL0J4tul X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.923,Hydra:6.0.545,FMLib:17.11.122.1 definitions=2023-01-06_06,2023-01-06_01,2022-06-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 clxscore=1015 priorityscore=1501 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 adultscore=0 impostorscore=0 lowpriorityscore=0 mlxscore=0 mlxlogscore=990 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000 definitions=main-2301060095 X-Spam-Status: No, score=-11.8 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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 <gcc-patches.gcc.gnu.org> List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe> List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/> List-Post: <mailto:gcc-patches@gcc.gnu.org> List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help> List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe> From: Jiufu Guo via Gcc-patches <gcc-patches@gcc.gnu.org> Reply-To: Jiufu Guo <guojiufu@linux.ibm.com> Errors-To: gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org Sender: "Gcc-patches" <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org> |
Series |
rs6000: mark tieable between INT and FLOAT
|
|
Commit Message
Jiufu Guo
Jan. 6, 2023, 12:25 p.m. UTC
Hi, During discussing/review patches in maillist, we find more modes are tieable, e.g. DI<->DF. With some discussion, I drafted this patch to mark more tieable modes. Bootstrap and regtest pass on ppc64{,le}. Is this ok for trunk? BR, Jeff (Jiufu) gcc/ChangeLog: * config/rs6000/rs6000.cc (rs6000_modes_tieable_p): Mark more tieable modes. gcc/testsuite/ChangeLog: * g++.target/powerpc/pr102024.C: Updated. --- gcc/config/rs6000/rs6000.cc | 9 +++++++++ gcc/testsuite/g++.target/powerpc/pr102024.C | 3 ++- 2 files changed, 11 insertions(+), 1 deletion(-)
Comments
Hi, Gently Ping: https://gcc.gnu.org/pipermail/gcc-patches/2023-January/609504.html BR, Jeff (Jiufu) Jiufu Guo <guojiufu@linux.ibm.com> writes: > Hi, > > During discussing/review patches in maillist, we find more modes are > tieable, e.g. DI<->DF. With some discussion, I drafted this patch > to mark more tieable modes. > > Bootstrap and regtest pass on ppc64{,le}. > Is this ok for trunk? > > BR, > Jeff (Jiufu) > > gcc/ChangeLog: > > * config/rs6000/rs6000.cc (rs6000_modes_tieable_p): Mark more tieable > modes. > > gcc/testsuite/ChangeLog: > > * g++.target/powerpc/pr102024.C: Updated. > > --- > gcc/config/rs6000/rs6000.cc | 9 +++++++++ > gcc/testsuite/g++.target/powerpc/pr102024.C | 3 ++- > 2 files changed, 11 insertions(+), 1 deletion(-) > > diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc > index 6ac3adcec6b..3cb0186089e 100644 > --- a/gcc/config/rs6000/rs6000.cc > +++ b/gcc/config/rs6000/rs6000.cc > @@ -1968,6 +1968,15 @@ rs6000_modes_tieable_p (machine_mode mode1, machine_mode mode2) > if (ALTIVEC_OR_VSX_VECTOR_MODE (mode2)) > return false; > > + /* SFmode format (IEEE DP) in register would not as required, > + So SFmode is restrict here. */ > + if (GET_MODE_CLASS (mode1) == MODE_FLOAT > + && GET_MODE_CLASS (mode2) == MODE_INT) > + return GET_MODE_SIZE (mode2) == UNITS_PER_FP_WORD && mode1 != SFmode; > + if (GET_MODE_CLASS (mode1) == MODE_INT > + && GET_MODE_CLASS (mode2) == MODE_FLOAT) > + return GET_MODE_SIZE (mode1) == UNITS_PER_FP_WORD && mode2 != SFmode; > + > if (SCALAR_FLOAT_MODE_P (mode1)) > return SCALAR_FLOAT_MODE_P (mode2); > if (SCALAR_FLOAT_MODE_P (mode2)) > diff --git a/gcc/testsuite/g++.target/powerpc/pr102024.C b/gcc/testsuite/g++.target/powerpc/pr102024.C > index 769585052b5..27d2dc5e80b 100644 > --- a/gcc/testsuite/g++.target/powerpc/pr102024.C > +++ b/gcc/testsuite/g++.target/powerpc/pr102024.C > @@ -5,7 +5,8 @@ > // Test that a zero-width bit field in an otherwise homogeneous aggregate > // generates a psabi warning and passes arguments in GPRs. > > -// { dg-final { scan-assembler-times {\mstd\M} 4 } } > +// { dg-final { scan-assembler-times {\mmtvsrd\M} 4 { target has_arch_pwr8 } } } > +// { dg-final { scan-assembler-times {\mstd\M} 4 { target { ! has_arch_pwr8 } } } } > > struct a_thing > {
Hi, I would ping this patch for stage1: https://gcc.gnu.org/pipermail/gcc-patches/2023-January/609504.html BR, Jeff (Jiufu) Jiufu Guo via Gcc-patches <gcc-patches@gcc.gnu.org> writes: > Hi, > > Gently Ping: > https://gcc.gnu.org/pipermail/gcc-patches/2023-January/609504.html > > BR, > Jeff (Jiufu) > > > Jiufu Guo <guojiufu@linux.ibm.com> writes: > >> Hi, >> >> During discussing/review patches in maillist, we find more modes are >> tieable, e.g. DI<->DF. With some discussion, I drafted this patch >> to mark more tieable modes. >> >> Bootstrap and regtest pass on ppc64{,le}. >> Is this ok for trunk? >> >> BR, >> Jeff (Jiufu) >> >> gcc/ChangeLog: >> >> * config/rs6000/rs6000.cc (rs6000_modes_tieable_p): Mark more tieable >> modes. >> >> gcc/testsuite/ChangeLog: >> >> * g++.target/powerpc/pr102024.C: Updated. >> >> --- >> gcc/config/rs6000/rs6000.cc | 9 +++++++++ >> gcc/testsuite/g++.target/powerpc/pr102024.C | 3 ++- >> 2 files changed, 11 insertions(+), 1 deletion(-) >> >> diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc >> index 6ac3adcec6b..3cb0186089e 100644 >> --- a/gcc/config/rs6000/rs6000.cc >> +++ b/gcc/config/rs6000/rs6000.cc >> @@ -1968,6 +1968,15 @@ rs6000_modes_tieable_p (machine_mode mode1, machine_mode mode2) >> if (ALTIVEC_OR_VSX_VECTOR_MODE (mode2)) >> return false; >> >> + /* SFmode format (IEEE DP) in register would not as required, >> + So SFmode is restrict here. */ >> + if (GET_MODE_CLASS (mode1) == MODE_FLOAT >> + && GET_MODE_CLASS (mode2) == MODE_INT) >> + return GET_MODE_SIZE (mode2) == UNITS_PER_FP_WORD && mode1 != SFmode; >> + if (GET_MODE_CLASS (mode1) == MODE_INT >> + && GET_MODE_CLASS (mode2) == MODE_FLOAT) >> + return GET_MODE_SIZE (mode1) == UNITS_PER_FP_WORD && mode2 != SFmode; >> + >> if (SCALAR_FLOAT_MODE_P (mode1)) >> return SCALAR_FLOAT_MODE_P (mode2); >> if (SCALAR_FLOAT_MODE_P (mode2)) >> diff --git a/gcc/testsuite/g++.target/powerpc/pr102024.C b/gcc/testsuite/g++.target/powerpc/pr102024.C >> index 769585052b5..27d2dc5e80b 100644 >> --- a/gcc/testsuite/g++.target/powerpc/pr102024.C >> +++ b/gcc/testsuite/g++.target/powerpc/pr102024.C >> @@ -5,7 +5,8 @@ >> // Test that a zero-width bit field in an otherwise homogeneous aggregate >> // generates a psabi warning and passes arguments in GPRs. >> >> -// { dg-final { scan-assembler-times {\mstd\M} 4 } } >> +// { dg-final { scan-assembler-times {\mmtvsrd\M} 4 { target has_arch_pwr8 } } } >> +// { dg-final { scan-assembler-times {\mstd\M} 4 { target { ! has_arch_pwr8 } } } } >> >> struct a_thing >> {
Gentle ping... Jiufu Guo via Gcc-patches <gcc-patches@gcc.gnu.org> writes: > Hi, > > I would ping this patch for stage1: > https://gcc.gnu.org/pipermail/gcc-patches/2023-January/609504.html > > BR, > Jeff (Jiufu) > > Jiufu Guo via Gcc-patches <gcc-patches@gcc.gnu.org> writes: > >> Hi, >> >> Gently Ping: >> https://gcc.gnu.org/pipermail/gcc-patches/2023-January/609504.html >> >> BR, >> Jeff (Jiufu) >> >> >> Jiufu Guo <guojiufu@linux.ibm.com> writes: >> >>> Hi, >>> >>> During discussing/review patches in maillist, we find more modes are >>> tieable, e.g. DI<->DF. With some discussion, I drafted this patch >>> to mark more tieable modes. >>> >>> Bootstrap and regtest pass on ppc64{,le}. >>> Is this ok for trunk? >>> >>> BR, >>> Jeff (Jiufu) >>> >>> gcc/ChangeLog: >>> >>> * config/rs6000/rs6000.cc (rs6000_modes_tieable_p): Mark more tieable >>> modes. >>> >>> gcc/testsuite/ChangeLog: >>> >>> * g++.target/powerpc/pr102024.C: Updated. >>> >>> --- >>> gcc/config/rs6000/rs6000.cc | 9 +++++++++ >>> gcc/testsuite/g++.target/powerpc/pr102024.C | 3 ++- >>> 2 files changed, 11 insertions(+), 1 deletion(-) >>> >>> diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc >>> index 6ac3adcec6b..3cb0186089e 100644 >>> --- a/gcc/config/rs6000/rs6000.cc >>> +++ b/gcc/config/rs6000/rs6000.cc >>> @@ -1968,6 +1968,15 @@ rs6000_modes_tieable_p (machine_mode mode1, machine_mode mode2) >>> if (ALTIVEC_OR_VSX_VECTOR_MODE (mode2)) >>> return false; >>> >>> + /* SFmode format (IEEE DP) in register would not as required, >>> + So SFmode is restrict here. */ >>> + if (GET_MODE_CLASS (mode1) == MODE_FLOAT >>> + && GET_MODE_CLASS (mode2) == MODE_INT) >>> + return GET_MODE_SIZE (mode2) == UNITS_PER_FP_WORD && mode1 != SFmode; >>> + if (GET_MODE_CLASS (mode1) == MODE_INT >>> + && GET_MODE_CLASS (mode2) == MODE_FLOAT) >>> + return GET_MODE_SIZE (mode1) == UNITS_PER_FP_WORD && mode2 != SFmode; >>> + >>> if (SCALAR_FLOAT_MODE_P (mode1)) >>> return SCALAR_FLOAT_MODE_P (mode2); >>> if (SCALAR_FLOAT_MODE_P (mode2)) >>> diff --git a/gcc/testsuite/g++.target/powerpc/pr102024.C b/gcc/testsuite/g++.target/powerpc/pr102024.C >>> index 769585052b5..27d2dc5e80b 100644 >>> --- a/gcc/testsuite/g++.target/powerpc/pr102024.C >>> +++ b/gcc/testsuite/g++.target/powerpc/pr102024.C >>> @@ -5,7 +5,8 @@ >>> // Test that a zero-width bit field in an otherwise homogeneous aggregate >>> // generates a psabi warning and passes arguments in GPRs. >>> >>> -// { dg-final { scan-assembler-times {\mstd\M} 4 } } >>> +// { dg-final { scan-assembler-times {\mmtvsrd\M} 4 { target has_arch_pwr8 } } } >>> +// { dg-final { scan-assembler-times {\mstd\M} 4 { target { ! has_arch_pwr8 } } } } >>> >>> struct a_thing >>> {
diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc index 6ac3adcec6b..3cb0186089e 100644 --- a/gcc/config/rs6000/rs6000.cc +++ b/gcc/config/rs6000/rs6000.cc @@ -1968,6 +1968,15 @@ rs6000_modes_tieable_p (machine_mode mode1, machine_mode mode2) if (ALTIVEC_OR_VSX_VECTOR_MODE (mode2)) return false; + /* SFmode format (IEEE DP) in register would not as required, + So SFmode is restrict here. */ + if (GET_MODE_CLASS (mode1) == MODE_FLOAT + && GET_MODE_CLASS (mode2) == MODE_INT) + return GET_MODE_SIZE (mode2) == UNITS_PER_FP_WORD && mode1 != SFmode; + if (GET_MODE_CLASS (mode1) == MODE_INT + && GET_MODE_CLASS (mode2) == MODE_FLOAT) + return GET_MODE_SIZE (mode1) == UNITS_PER_FP_WORD && mode2 != SFmode; + if (SCALAR_FLOAT_MODE_P (mode1)) return SCALAR_FLOAT_MODE_P (mode2); if (SCALAR_FLOAT_MODE_P (mode2)) diff --git a/gcc/testsuite/g++.target/powerpc/pr102024.C b/gcc/testsuite/g++.target/powerpc/pr102024.C index 769585052b5..27d2dc5e80b 100644 --- a/gcc/testsuite/g++.target/powerpc/pr102024.C +++ b/gcc/testsuite/g++.target/powerpc/pr102024.C @@ -5,7 +5,8 @@ // Test that a zero-width bit field in an otherwise homogeneous aggregate // generates a psabi warning and passes arguments in GPRs. -// { dg-final { scan-assembler-times {\mstd\M} 4 } } +// { dg-final { scan-assembler-times {\mmtvsrd\M} 4 { target has_arch_pwr8 } } } +// { dg-final { scan-assembler-times {\mstd\M} 4 { target { ! has_arch_pwr8 } } } } struct a_thing {