Message ID | Yd8T6AoTbTWC7jHs@toto.the-meissners.org |
---|---|
State | Committed |
Commit | 7d8011fa00fca283003c84e23a8ca66286f83dfa |
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 0E0873864837 for <patchwork@sourceware.org>; Wed, 12 Jan 2022 17:46:53 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0E0873864837 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1642009613; bh=uRXhMmBa8TM5CkBBixk6ky89+ehDudtQo6hGvBKWiVE=; h=Date:To:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=W0niG4ZhA0rp3zerEykYSFbUt27xCf511hlduPtKxAk+zzucDRccmyUHrgVqSesMW bQq4b5+Zkmxy2J75/1V0GqKByQglBlchM0Oz7X12kslZb87QrIF4dke+DxnEoLQpG3 ZrWHod3IIQGLeOYoUEpPED3Bm9U+OqiK2UqvgMAM= 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 9E0693858406 for <gcc-patches@gcc.gnu.org>; Wed, 12 Jan 2022 17:46:23 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 9E0693858406 Received: from pps.filterd (m0098413.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 20CFuhx0010600; Wed, 12 Jan 2022 17:46:22 GMT Received: from pps.reinject (localhost [127.0.0.1]) by mx0b-001b2d01.pphosted.com with ESMTP id 3dj22xj6eh-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 12 Jan 2022 17:46:21 +0000 Received: from m0098413.ppops.net (m0098413.ppops.net [127.0.0.1]) by pps.reinject (8.16.0.43/8.16.0.43) with SMTP id 20CHBlOX018670; Wed, 12 Jan 2022 17:46:21 GMT Received: from ppma02dal.us.ibm.com (a.bd.3ea9.ip4.static.sl-reverse.com [169.62.189.10]) by mx0b-001b2d01.pphosted.com with ESMTP id 3dj22xj6e2-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 12 Jan 2022 17:46:21 +0000 Received: from pps.filterd (ppma02dal.us.ibm.com [127.0.0.1]) by ppma02dal.us.ibm.com (8.16.1.2/8.16.1.2) with SMTP id 20CHfrcG030750; Wed, 12 Jan 2022 17:46:20 GMT Received: from b03cxnp07029.gho.boulder.ibm.com (b03cxnp07029.gho.boulder.ibm.com [9.17.130.16]) by ppma02dal.us.ibm.com with ESMTP id 3df28bkxq4-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 12 Jan 2022 17:46:20 +0000 Received: from b03ledav002.gho.boulder.ibm.com (b03ledav002.gho.boulder.ibm.com [9.17.130.233]) by b03cxnp07029.gho.boulder.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 20CHkJiv22086120 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 12 Jan 2022 17:46:19 GMT Received: from b03ledav002.gho.boulder.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id ED1C013605E; Wed, 12 Jan 2022 17:46:18 +0000 (GMT) Received: from b03ledav002.gho.boulder.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 6842E136067; Wed, 12 Jan 2022 17:46:18 +0000 (GMT) Received: from toto.the-meissners.org (unknown [9.77.149.10]) by b03ledav002.gho.boulder.ibm.com (Postfix) with ESMTPS; Wed, 12 Jan 2022 17:46:18 +0000 (GMT) Date: Wed, 12 Jan 2022 12:46:16 -0500 To: gcc-patches@gcc.gnu.org, Michael Meissner <meissner@linux.ibm.com>, Segher Boessenkool <segher@kernel.crashing.org>, David Edelsohn <dje.gcc@gmail.com>, Bill Schmidt <wschmidt@linux.ibm.com>, Peter Bergner <bergner@linux.ibm.com>, Will Schmidt <will_schmidt@vnet.ibm.com> Subject: [PATCH] Allow other languages to change long double format on PowerPC Message-ID: <Yd8T6AoTbTWC7jHs@toto.the-meissners.org> Mail-Followup-To: Michael Meissner <meissner@linux.ibm.com>, gcc-patches@gcc.gnu.org, Segher Boessenkool <segher@kernel.crashing.org>, David Edelsohn <dje.gcc@gmail.com>, Bill Schmidt <wschmidt@linux.ibm.com>, Peter Bergner <bergner@linux.ibm.com>, Will Schmidt <will_schmidt@vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-TM-AS-GCONF: 00 X-Proofpoint-GUID: 0SIOd5pYS62ZplCdrrXUs0NmByKLs3wj X-Proofpoint-ORIG-GUID: eklqMoDjd3tQaaW_IItljU_PQF7wzP0i X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.11.62.513 definitions=2022-01-12_04,2022-01-11_01,2021-12-02_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 mlxscore=0 phishscore=0 spamscore=0 impostorscore=0 mlxlogscore=999 priorityscore=1501 adultscore=0 lowpriorityscore=0 bulkscore=0 suspectscore=0 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000 definitions=main-2201120105 X-Spam-Status: No, score=-10.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_EF, GIT_PATCH_0, KAM_MANYTO, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) 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: Michael Meissner via Gcc-patches <gcc-patches@gcc.gnu.org> Reply-To: Michael Meissner <meissner@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 |
Allow other languages to change long double format on PowerPC
|
|
Commit Message
Michael Meissner
Jan. 12, 2022, 5:46 p.m. UTC
Allow other languages to change long double format on PowerPC. With Fortran adding support for changing the long double format, this patch removes the code that only allowed C/C++ to change the long double format for GLIBC 2.32 and later without a warning. I have bootstraped the compiler with this change and there where no regressesion. In addition, I have applied it to the power-ieee128 branch where the work is being done to add support for both types of 128-bit floating point on PowerPC. If I build a compiler from that branch, I can change the 128-bit floating point format at compile time. Can I check this patch into the trunk GCC compiler? gcc/ 2022-01-12 Michael Meissner <meissner@the-meissners.org> * config/rs6000/rs6000.c (rs6000_option_override_internal): Remove checks for only C/C++ front ends before allowing the long double format to change without a warning. --- gcc/config/rs6000/rs6000.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-)
Comments
On Wed, Jan 12, 2022 at 12:46:16PM -0500, Michael Meissner via Gcc-patches wrote: > Allow other languages to change long double format on PowerPC. > > With Fortran adding support for changing the long double format, this > patch removes the code that only allowed C/C++ to change the long double > format for GLIBC 2.32 and later without a warning. > > I have bootstraped the compiler with this change and there where no > regressesion. In addition, I have applied it to the power-ieee128 branch where > the work is being done to add support for both types of 128-bit floating point > on PowerPC. If I build a compiler from that branch, I can change the 128-bit > floating point format at compile time. > > Can I check this patch into the trunk GCC compiler? > > gcc/ > 2022-01-12 Michael Meissner <meissner@the-meissners.org> > > * config/rs6000/rs6000.c (rs6000_option_override_internal): Remove > checks for only C/C++ front ends before allowing the long double > format to change without a warning. This is already on the trunk, I've merged the power-ieee128 branch changes into trunk yesterday (r12-6491 through r12-6508 commits inclusive). This patch in particular as r12-6503-g7d8011fa00fca283003c84e23a8ca66286f83dfa. Jakub
On Wed, Jan 12, 2022 at 06:50:04PM +0100, Jakub Jelinek wrote: > On Wed, Jan 12, 2022 at 12:46:16PM -0500, Michael Meissner via Gcc-patches wrote: > > Allow other languages to change long double format on PowerPC. > > > > With Fortran adding support for changing the long double format, this > > patch removes the code that only allowed C/C++ to change the long double > > format for GLIBC 2.32 and later without a warning. > > > > I have bootstraped the compiler with this change and there where no > > regressesion. In addition, I have applied it to the power-ieee128 branch where > > the work is being done to add support for both types of 128-bit floating point > > on PowerPC. If I build a compiler from that branch, I can change the 128-bit > > floating point format at compile time. > > > > Can I check this patch into the trunk GCC compiler? > > > > gcc/ > > 2022-01-12 Michael Meissner <meissner@the-meissners.org> > > > > * config/rs6000/rs6000.c (rs6000_option_override_internal): Remove > > checks for only C/C++ front ends before allowing the long double > > format to change without a warning. > > This is already on the trunk, I've merged the power-ieee128 branch changes > into trunk yesterday (r12-6491 through r12-6508 commits inclusive). > This patch in particular as r12-6503-g7d8011fa00fca283003c84e23a8ca66286f83dfa. Thanks. I didn't notice that the patches had been merged into the trunk.
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 319182e94d9..0e3481c8327 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -4221,13 +4221,11 @@ rs6000_option_override_internal (bool global_init_p) if (rs6000_ieeequad != TARGET_IEEEQUAD_DEFAULT && TARGET_LONG_DOUBLE_128) { /* Determine if the user can change the default long double type at - compilation time. Only C and C++ support this, and you need GLIBC - 2.32 or newer. Only issue one warning. */ + compilation time. You need GLIBC 2.32 or newer to be able to + change the long double type. Only issue one warning. */ static bool warned_change_long_double; - if (!warned_change_long_double - && (!glibc_supports_ieee_128bit () - || (!lang_GNU_C () && !lang_GNU_CXX ()))) + if (!warned_change_long_double && !glibc_supports_ieee_128bit ()) { warned_change_long_double = true; if (TARGET_IEEEQUAD)