Re: [Patch, fortran] PR 49074 ICE on defined assignment with class arrays.

2013-06-15 Thread Tobias Burnus
Mikael Morin wrote: Dominique noticed that the patch also fixed PR56136 whose test is very close to the one of PR49074. This made me notice that while the PR56136 test should use a temporary (and does), the PR49074 one shouldn't. That is fixed with the attached patch. Then the ICE which was fixe

Re: [Patch, fortran] PR 49074 ICE on defined assignment with class arrays.

2013-06-15 Thread Mikael Morin
Hello, Dominique noticed that the patch also fixed PR56136 whose test is very close to the one of PR49074. This made me notice that while the PR56136 test should use a temporary (and does), the PR49074 one shouldn't. That is fixed with the attached patch. Then the ICE which was fixed by the previ

Re: [Patch, fortran] PR 49074 ICE on defined assignment with class arrays.

2013-06-12 Thread Tobias Burnus
Hello Mikael, Mikael Morin wrote: Regression tested on x86_64-unknown-linux-gnu. OK for trunk? OK - looks good to me. The test case is also nice and a bit tricky, I tried it with three compilers: Two segfaulted at run time and only one passed the test. Tobias

[Patch, fortran] PR 49074 ICE on defined assignment with class arrays.

2013-06-12 Thread Mikael Morin
Hello, this is a fix for PR49074, where the temporary created by gfc_conv_elemental_dependencies was leading to an ICE because it didn't have the array reference expected by the scalarization code. There was a bypass in gfc_conv_procedure_call avoiding exactly this problem, but it is not reached