[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-05 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

Paul Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Paul Thomas  ---
Thanks for flagging the problem up and confirming the fix so promptly.

Fixed.

Paul

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-05 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

--- Comment #6 from Paul Thomas  ---
Author: pault
Date: Fri Oct  5 07:01:57 2018
New Revision: 264862

URL: https://gcc.gnu.org/viewcvs?rev=264862=gcc=rev
Log:
2018-10-05  Paul Thomas  

PR fortran/87487
* trans-decl.c (gfc_get_symbol_decl): Make sure that deferred
character length pointer initializer has the right type to fix
problem with deferred_character_24.f90 on big endian.


Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/trans-decl.c

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-04 Thread seurer at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

--- Comment #5 from seurer at gcc dot gnu.org ---
I tried it on a powerpc64 BE machine and it worked there too.

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-04 Thread paul.richard.thomas at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

--- Comment #4 from paul.richard.thomas at gmail dot com  ---
Dear Rainer,

That's a relief!  Thank you for the fast response. I will commit tonight.

Paul
On Thu, 4 Oct 2018 at 12:04, ro at CeBiTec dot Uni-Bielefeld.DE
 wrote:
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487
>
> --- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE  Uni-Bielefeld.DE> ---
> > --- Comment #2 from Paul Thomas  ---
> > Created attachment 44781
> >   --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44781=edit
> > Possible fix
> >
> > Does this do it for you?
>
> As a quick test, I just rebuilt f951 and ran the single testcase: it
> PASSes now for both 32 and 64-bit.
>
> Thanks.
> Rainer
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
> You are the assignee for the bug.

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-04 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE  ---
> --- Comment #2 from Paul Thomas  ---
> Created attachment 44781
>   --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44781=edit
> Possible fix
>
> Does this do it for you?

As a quick test, I just rebuilt f951 and ran the single testcase: it
PASSes now for both 32 and 64-bit.

Thanks.
Rainer

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-04 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

Paul Thomas  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |pault at gcc dot gnu.org

--- Comment #2 from Paul Thomas  ---
Created attachment 44781
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44781=edit
Possible fix

Does this do it for you?

I am very surprised to run into an endian problem with this.

Cheers

Paul

[Bug testsuite/87487] New test case gfortran.dg/deferred_character_24.f90 in r264721 fails on big endian

2018-10-04 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87487

Rainer Orth  changed:

   What|Removed |Added

 Target|powerpc64-unknown-linux-gnu |powerpc64-unknown-linux-gnu
   ||, sparc-sun-solaris2.*
 Status|UNCONFIRMED |NEW
   Last reconfirmed||2018-10-04
 CC||ro at gcc dot gnu.org
   Host|powerpc64-unknown-linux-gnu |
   Target Milestone|--- |9.0
 Ever confirmed|0   |1
  Build|powerpc64-unknown-linux-gnu |

--- Comment #1 from Rainer Orth  ---
I'm seeing the same on 64-bit Solaris/SPARC.