Re: [Patch, fortran] PR66089 fix elemental dependency mishandling

2016-02-03 Thread Paul Richard Thomas
Dear Mikael, The patch is OK for trunk. A small niggle: Although present in the original testcase, 'a' is unused. I am not in a position to find out for myself, right now, but does the testcase of comment #10 work with this patch? Thanks for the patch Paul On 1 February 2016 at 23:07, Mikael

Re: [Patch, fortran] PR66089 fix elemental dependency mishandling

2016-02-03 Thread Mikael Morin
Le 03/02/2016 14:00, Paul Richard Thomas a écrit : Dear Mikael, The patch is OK for trunk. A small niggle: Although present in the original testcase, 'a' is unused. Indeed, I'll remove it. I am not in a position to find out for myself, right now, but does the testcase of comment #10 work

[Patch, fortran] PR66089 fix elemental dependency mishandling

2016-02-01 Thread Mikael Morin
Hello, this is about the case c(:) = elemental_func(c(1), ...) where as a result of a trunk change, only a reference to c(1) is saved to a temporary variable, instead of its value. The fix tries to save the amount of copying as much as possible by detecting the above case. Technically