From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 720B7385C6DE; Fri, 21 Jul 2023 05:30:53 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 720B7385C6DE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1689917453; bh=4yLRna6oMh7Is9qEdlYJ/87nt+3TvVxhh6lTb+P3nq8=; h=From:To:Subject:Date:In-Reply-To:References:From; b=toH4qmQ9Mm/2vsSVi/92uYvcN+tp9vAE8C+AbQlCMk58+oJQOOK/qgNWh3HbcuHQ+ hSGdh82Trfcvx4G2OPnARRiG8vP5QtD28LtP3g15UZGDgD2GXT3u392lqSI9Aq8Bhl W1JKEifxkky4hDKEZhSs0gc9hQ/V3ad2kxnJUCWU= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug libfortran/110759] [14 Regression] IEEE Fortran change broke RISC-V linux build Date: Fri, 21 Jul 2023 05:30:53 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: libfortran X-Bugzilla-Version: 14.0 X-Bugzilla-Keywords: build X-Bugzilla-Severity: normal X-Bugzilla-Who: cvs-commit at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: pinskia at gcc dot gnu.org X-Bugzilla-Target-Milestone: 14.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D110759 --- Comment #9 from CVS Commits --- The trunk branch has been updated by Andrew Pinski : https://gcc.gnu.org/g:f32518726ee8e836d12d49aec8432679fcc42503 commit r14-2695-gf32518726ee8e836d12d49aec8432679fcc42503 Author: Andrew Pinski Date: Fri Jul 21 02:26:09 2023 +0000 libfortran: Fix build for targets that don't have 10byte or 16 byte floating point So the problem here is EXPAND_INTER_MACRO_16 expands to nothing if 16 b= yte FP does not exist but we still add a comma after it and that causes a build failure. The same is true for EXPAND_INTER_MACRO_10 too. Committed as obvious after a bootstrap and test on x86_64-linux-gnu and aarch64-linux-gnu. libgfortran/ChangeLog: PR libfortran/110759 * ieee/ieee_arithmetic.F90 (COMP_INTERFACE): Remove the comma after EXPAND_INTER_MACRO_16 and EXPAND_INTER_MACRO_10. (EXPAND_INTER_MACRO_16): Add comma here if 16 byte fp exist. (EXPAND_INTER_MACRO_10): Likewise.=