[Bug d/90130] gdc.test/runnable/test12.d FAILs

2019-04-24 Thread ro at CeBiTec dot Uni-Bielefeld.DE 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90130

--- Comment #4 from ro at CeBiTec dot Uni-Bielefeld.DE  ---
> --- Comment #3 from Iain Buclaw  ---
> I think it should be done in r270485.

Indeed.  It works fine on i386-pc-solaris2.11 and sparc-sun-solaris2.11
with gas.  I do get BUS errors in
gcc.unwind.pe.read_encoded_value_with_base on Solaris 11/SPARC when
using /bin/as, but that's unrelated: there are several more failures
that don't occur with gas.

[Bug d/90130] gdc.test/runnable/test12.d FAILs

2019-04-23 Thread ibuclaw at gdcproject dot org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90130

Iain Buclaw  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Iain Buclaw  ---
I think it should be done in r270485.

[Bug d/90130] gdc.test/runnable/test12.d FAILs

2019-04-21 Thread ibuclaw at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90130

--- Comment #2 from ibuclaw at gcc dot gnu.org ---
Author: ibuclaw
Date: Sun Apr 21 20:26:12 2019
New Revision: 270485

URL: https://gcc.gnu.org/viewcvs?rev=270485=gcc=rev
Log:
PR d/90130
d/dmd: Merge upstream dmd 065fbd452

Fixes endian bug in CTFE, and corrects tests in the D2 testsuite that
failed on big endian targets.

Initial patch by Robin Dapp.

Reviewed-on: https://github.com/dlang/dmd/pull/9665

Modified:
trunk/gcc/d/dmd/MERGE
trunk/gcc/d/dmd/constfold.c
trunk/gcc/testsuite/gdc.test/runnable/mars1.d
trunk/gcc/testsuite/gdc.test/runnable/test12.d
trunk/gcc/testsuite/gdc.test/runnable/test23.d

[Bug d/90130] gdc.test/runnable/test12.d FAILs

2019-04-20 Thread ibuclaw at gdcproject dot org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90130

--- Comment #1 from Iain Buclaw  ---
(In reply to Rainer Orth from comment #0)
> The gdc.test/runnable/test12.d test FAILs on Solaris 11/SPARC with Robin's
> big-endian
> patches applied:
> 
> FAIL: gdc.test/runnable/test12.d -finline-functions -funittest -g  
> execution test
> 
> core.exception.AssertError@runnable/test12.d(630): Assertion failure
> 

-- snip --

> (gdb) p/x *(int *)
> $9 = 0x1020304
> 
> I don't fully see why yet, however all this strongly argues that this part of
> testcase is bogus: you cannot pass a struct to sprintf whose format string
> expects an int.

I'm not sure the reason for the test - there's no upstream bugzilla reference,
and it was added 10 years or so back in one big bulk import.

Rather than removing outright, I guess using *cast(int*) as you've been doing
in gdb is probably the path of least resistance.

[Bug d/90130] gdc.test/runnable/test12.d FAILs

2019-04-17 Thread ro at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90130

Rainer Orth  changed:

   What|Removed |Added

   Target Milestone|--- |9.0