On Fri, Nov 4, 2011 at 2:02 AM, Mikael Morin <mikael.mo...@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 >
