From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugzilla@gcc.gnu.org>
Received: by sourceware.org (Postfix, from userid 48)
	id 1657F3858025; Tue, 16 Jan 2024 10:22:39 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 1657F3858025
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1705400560;
	bh=ID8KWt+0T5/7HUnzZe/LkInijh6PZiraxp9gEmGYMKA=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=iqbEl+vt+qsBa32tyOeKlXu+UIPCbAsKv3k42jd0j1FToivJU18U0/XENei6IEmUc
	 zRYio2prSd7/7C6d1Kf2HyoJ+7qu4LPJe9paLVR9gBfZA75H7XM1YBpUktbhTyG+dA
	 I+iu/np79kcUmjz+b9QB1MhdS+cEki/iGNWwkPp4=
From: "gaiusmod2 at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug modula2/111956] [14 Regression] Many powerpc platforms do _not_
 have support for IEEE754 long double
Date: Tue, 16 Jan 2024 10:22:36 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: modula2
X-Bugzilla-Version: 14.0
X-Bugzilla-Keywords: build
X-Bugzilla-Severity: normal
X-Bugzilla-Who: gaiusmod2 at gmail dot com
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: gaius at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 14.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-111956-4-TwYwCVwgiC@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-111956-4@http.gcc.gnu.org/bugzilla/>
References: <bug-111956-4@http.gcc.gnu.org/bugzilla/>
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: <gcc-bugs.sourceware.org>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D111956

--- Comment #16 from gaiusmod2 at gmail dot com ---
"rguenth at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> writes:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D111956
>
> --- Comment #15 from Richard Biener <rguenth at gcc dot gnu.org> ---
> (In reply to Gaius Mulley from comment #14)
>> Ah apologies, is it best that I revert:
>>=20
>> https://gcc.gnu.org/git/gitweb.cgi?p=3Dgcc.git;
>> h=3D81d5ca0b9b8431f1bd7a5ec8a2c94f04bb0cf032
>>=20
>> happy to do this in the morning.
>
> I think it might be better to define M2C_LONGREAL_FLOAT128/M2C_LONGREAL_P=
PC64LE
> (whatever they exactly indicate) in terms of the existing
>
> --with-long-double-128
> --with-long-double-format
>
> aka the TARGET_DEFAULT_LONG_DOUBLE_128 that's put into the config plus
> with_long_double_format (I think that causes TARGET_IEEEQUAD_DEFAULT
> to be defined to 1/0, but only for ppc, via config.gcc and
> tm_defines).

yes thanks for the hints this sounds good.  I'll pursue this line for a fix.

> I can't say whether it's better to revert or disable/fix as I can't say
> how this for example affects the M2 ABI (like if there was any 'long doub=
le'
> before this change and what effective type this used).

Prior to the patch it used the default C long double type but many of
the intrinsic functions were broken.  Currently with the patch gcc120
has no regression test failures and gcc135 (yesterday with manual
configure intervention) was at 96 failures in the m2 testsuite.

I suspect configure confusion and hence using
TARGET_DEFAULT_LONG_DOUBLE_128 and TARGET_IEEEQUAD_DEFAULT should
resolve it.=