Possible patch for 62242

2015-07-31 Thread Louis Krupp
As far as I can tell, the problem in 62242 (and possibly 62246) is with a string array constructor trying to deal with an array element whose value is a character function that is described in an interface block and which has an assumed-length result. I can't claim more than a superficial under

Possible patch for fortran/65766

2015-07-31 Thread Louis Krupp
The problem is with substrings of allocatable string components of derived types. The code seems to be trying to get the string length from typespec of the derived type variable instead of from the component. The attached patch gets the component typespec from the reference chain. I don't unde

Patch for fortran/62536 and fortran/66175

2015-07-31 Thread Louis Krupp
s. Index: ChangeLog === --- ChangeLog (revision 226452) +++ ChangeLog (working copy) @@ -1877,6 +1877,13 @@ * interface.c (is_procptr_result): New function to check if an expression is a procedure-pointer result. (compare_actual_formal): Use it. + +2015_07_31 Louis Krup

Re: Re: Re: Ping: New suggested patch for pr 62242 & pr 52332

2015-10-01 Thread Louis Krupp
hemselves. The textual part of your comment is >perfectly clear. > >Many thanks for the patch. > >Paul > >PS Do you now have all the paperwork to commit the patch yourself? > >On 29 September 2015 at 09:36, Louis Krupp wrote: >> Paul, >> >>

[PATCH COMMITTED] MAINTAINERS (Write After Approval): Add myself.

2015-10-02 Thread Louis Krupp
I have just added myself. Index: ChangeLog === --- ChangeLog (revision 228379) +++ ChangeLog (working copy) @@ -1,3 +1,7 @@ +2015-10-02 Louis Krupp + + * MAINTAINERS (Write After Approval): Add myself. + 2015-09-20 Kai

[Fortran, committed] Fix ICE in string array constructor (pr 62242 & pr 52332)

2015-10-02 Thread Louis Krupp
Revision 228368... Louis

Re: Possible patch for fortran/65766

2015-10-03 Thread Louis Krupp
I'm ready to check it in. Louis On Sat, 03 Oct 2015 07:00:56 -0700 Dominique d'Humières wrote > AFAICT this patch has been approved by FX at > https://gcc.gnu.org/ml/fortran/2015-07/msg00168.html. Any reason to not > commit it? > > Dominique > >

[Fortran, committed] Fix segfault on substring of derived type component (pr 65766)

2015-10-06 Thread Louis Krupp
Revision 228551... Louis

Possible patch for PR fortran/67806

2015-10-11 Thread Louis Krupp
The problem involves a derived type with a character component declared CHARACTER(NULL()) or CHARACTER(NULL(n)), where mold argument n is an integer pointer. I might be missing something, but I'm not sure there's a point to having a character variable whose length is the target of a null pointe

Re: Possible patch for PR fortran/67806

2015-10-12 Thread Louis Krupp
On Mon, 12 Oct 2015 08:41:43 -0700 Steve Kargl wrote > On Sun, Oct 11, 2015 at 10:18:48PM -0700, Louis Krupp wrote: > > The problem involves a derived type with a character component declared > > CHARACTER(NULL()) or CHARACTER(NULL(n)), where mold argument n is a

Possible patch for PR fortran/66056

2015-10-25 Thread Louis Krupp
The problem: Statement labels within a type declaration are put in the statement label tree belonging to the type declaration's namespace's (instead of the current namespace). When the line is otherwise empty and an error is issued, gfc_free_st_label tries to delete the label from the label tr

Re: Possible patch for PR fortran/66056

2015-10-26 Thread Louis Krupp
=== --- gcc/fortran/ChangeLog (revision 229307) +++ gcc/fortran/ChangeLog (working copy) @@ -1,3 +1,14 @@ +2015-10-26 Louis Krupp + + PR fortran/66056 + * fortran.h: Include namespace pointer in statement label + structure. + * symbol.c (gfc_get_st_label): Store

[Fortran, committed] Statement label on empty line in derived type declaration (PR 66056)

2015-10-26 Thread Louis Krupp
Revision 229390... Louis

Fortran, committed: Init character with non-char constructor (pr 83092)

2017-12-27 Thread Louis Krupp
Fixed in revision 256011.

Fortran, committed: ICE on CLASS(*) function result (pr 82257)

2018-01-15 Thread Louis Krupp
Fixed in revision 256720.

Fortran, committed: Forall-with-temporary problems(pr 50069 and pr 55086).

2017-01-18 Thread Louis Krupp
Fixed in revision 244601.

Re: Re: Fortran, committed: Forall-with-temporary problems(pr 50069 and pr 55086).

2017-01-19 Thread Louis Krupp
hu, 19 Jan 2017 04:52:54 -0800 Christophe Lyon wrote >Hi, > > >On 18 January 2017 at 22:45, Louis Krupp wrote: >> Fixed in revision 244601. >> > >I've noticed a new failure on arm/aarch64: > compiler driver --help=fortran option(s): "^

Fortran, committed: ICE out of memory on displaced implicit character (pr 69963)

2016-09-15 Thread Louis Krupp
Fixed in revision 240168.

Fortran, committed: segfault with allocate and stat for derived types with default initialization (pr 68078)

2016-09-17 Thread Louis Krupp
Fixed in revision 240219.

Re: Fortran, committed: segfault with allocate and stat for derived types with default initialization (pr 68078)

2016-09-18 Thread Louis Krupp
Two possibilities: 1. malloc() doesn't silently return NULL on Darwin when it runs out of memory; it always generates an error message. 2. setrlimit() doesn't work the same on Darwin as it does on Linux, and the test program is hitting a system limit. It so happens that when I first looked a

Re: Fortran, committed: segfault with allocate and stat for derived types with default initialization (pr 68078)

2016-09-21 Thread Louis Krupp
least one system, we can call ! it good. Louis On Mon, 19 Sep 2016 02:35:21 -0700 Christophe Lyon wrote > Hi, > > > On 18 September 2016 at 21:19, Louis Krupp wrote: > > Two possibilities: > > > > 1. malloc() doesn't silently

Fortran, committed: ICE on missing parameter value for initialisation (segfault) (pr 66107)

2016-09-22 Thread Louis Krupp
Fixed in revision 240341.

gcc build problem (i386.c) -- missing declaration

2016-09-22 Thread Louis Krupp
cc list as far as I know.) Louis Krupp

Re: gcc build problem (i386.c) -- missing declaration

2016-09-29 Thread Louis Krupp
My target was gfortran. In any case, someone else fixed this problem. Louis On Thu, 29 Sep 2016 11:10:15 -0700 Jeff Law wrote > On 09/22/2016 04:52 PM, Louis Krupp wrote: > > As of revision 240383 , i386.c isn't compiling. The errors are: > > >

Fortran, committed: ICE on using implicit character instead of implicit none etc. (pr 67524)

2016-10-05 Thread Louis Krupp
Fixed in revision 240797.

Re: Possible patch for fortran/57910

2016-10-06 Thread Louis Krupp
Dominique, Vous avez raison. I attached the wrong patch. I've resent the message with the correct patch. I tried to make pr69955.f90 run only on 64-bit Linux: ! { dg-do run { target x86_64-*-linux* } } I'm not sure there's a portable way to query virtual memory usage, and testing this on on

Re: Possible patch for fortran/57910

2016-10-06 Thread Louis Krupp
I've attached an updated patch for pr69955. It works just as you said. Please let me know if this or my patch for pr57910 is OK to check in. Louis On Thu, 06 Oct 2016 14:30:29 -0700 Dominique d'Humières wrote > > > Le 6 oct. 2016 à 19:35, L

Fortran, committed: ICE (segfault) with deferred-length strings (pr 57910)

2016-10-06 Thread Louis Krupp
Fixed in revision 240850.

Fortran, committed: Memory leak with array constructor and derived type (pr 69955).

2016-10-06 Thread Louis Krupp
Fixed in revision 240851.

Possible patch for pr62242

2015-09-09 Thread Louis Krupp
=== --- gcc/fortran/ChangeLog (revision 227571) +++ gcc/fortran/ChangeLog (working copy) @@ -1,3 +1,12 @@ +2015-09-08 Louis Krupp + + PR fortran/62242 + * trans-array.c (get_array_ctor_all_strlen): Don't store length + tree po

Possible patch for pr62242 -- follow-up

2015-09-14 Thread Louis Krupp
Would anyone like me to spend some more time on this and perhaps clear up some of the TODO items? (Unlike some of you, I'm retired. I have time for this.) Louis == == == == == == Forwarded message == == == == == == >From : Louis Krupp To : "gcc-patches","fortran"

Re: Possible patch for fortran/55086

2016-11-06 Thread Louis Krupp
> > I would not be surprised if there were a better way to do this. > > > > When I tweak gfc_trans_forall_1() to force usage of a temporary > > for all assignments, all tests pass with the exception > > of forall_8.f90, and the whole point of that test is making > > sure that this particular forall *doesn't* use a temporary. > > > > Louis Krupp > > >