[Patch, fortran] [00/66] PR fortran/43829 Inline sum and product (AKA scalarization of reductions)

Richard Guenther richard.guenther@gmail.com
Fri Nov 4 09:39:00 GMT 2011


On Fri, Nov 4, 2011 at 2:02 AM, Mikael Morin <mikael.morin@sfr.fr> wrote:
> On Tuesday 01 November 2011 22:07:48 Paul Richard Thomas wrote:
>> The only, slight worry that I have is that it is going to make Richi's
>> middle end scalarization nearly impossible to use for gfortran.
>> However, the enhanced capability that this patch brings makes it a
>> worthy addition to gfortran.
>>
> I think that Richi's middle-end arrays and gfortran's scalarizer are more or
> less incompatible, regardless of this patch. And if they are made to coexist
> side by side at some point, this patch won't make it either better/easier or
> worse/harder IMHO.

Indeed.  In the ideal form the current scalarizer could defer scalarizing
innermost loop (nests) to the middle-end, my hackish patches from earlier
this year of course were just hacks (to eventually get some interest from
you folks).  But of course not even the middle-end parts are completely
ready (partly also due to lack of good test coverage possibility - a chicken
and egg problem :/).

So I think improving the Fortran scalarizer is time that is very well spent.

Thanks,
Richard.

>
>> OK for trunk.
>>
> Thanks for the review!
> Committed as follows.
>
> Mikael
>
> patch revision
> ============
> 01    180842
> 02    180843
> 03    180844
> 04    180846
> 05    180847
> 06    180848
> 08    180849
> 07    180850
> 09    180851
> 10    180852
> 11    180853
> 12    180855
> 13    180856
> 14    180857
> 15    180858
> 16    180859
> 17    180860
> 18    180861
> 19    180862
> 20    180864
> 21    180865
> 22    180866
> 23    180867
> 24    180868
> 25    180869
> 26    180870
> 27    180872
> 28    180873
> 29    180875
> 30    180877
> 31    180880
> 32    180881
> 33    180882
> 34    180883
> 35    180884
> 36    180885
> 37    180886
> 38    180887
> 39    180888
> 40    180889
> 41    180890
> 42    180891
> 43    480892
> 44    180893
> 45    180894
> 46    180895
> 47    180897
> 48    180898
> 49    180899
> 50    180900
> 51    180901
> 52    180902
> 53    180903
> 54    180904
> 55    180905
> 56    180906
> 57    180907
> 58    180908
> 59    180909
> 60    180910
> 61    180911
> 62    180913
> 63    180917
> 64    180918
> 65    180919
> 66    180920
> tests 180922
>



More information about the Gcc-patches mailing list