This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH], PowerPC long double transition patches, v2, Patch #6 (fix long double tests for -mno-float128)


On Thu, Jun 21, 2018 at 06:12:52PM -0500, Segher Boessenkool wrote:
> On Wed, Jun 20, 2018 at 10:49:41AM -0400, Michael Meissner wrote:
> > These patches fix the tests in the testsuite that check whether -mno-float128
> > works properly.  In these cases, I explicitly run them with long double being
> > set to IBM extended double.
> 
> So what happened without this patch?

To give context, it was this patch you were asking about:

2018-06-20  Michael Meissner  <meissner@linux.ibm.com>

	* gcc.target/powerpc/float128-3.c: Change tests that test for
	errors in enabling/disable IEEE 128-bit floating point use the
	-mabi=ibmlongdouble to force long double to be IBM extended.
	* gcc.target/powerpc/float128-5.c: Likewise.
	* gcc.target/powerpc/float128-mix.c: Likewise.

float128-3.c and float128-5.c both issues no error (because they both test if
__FLOAT128__ defined even if you use the -mno-float128 option), instead of the
two errors that the test checks for.

float128-mix.c also issues no error because with -mabi=ieeelongdouble, it is
legal to allow __float128 and long double to be combined.

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.ibm.com, phone: +1 (978) 899-4797


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]