> On Mon, 27 May 2019, Jan Hubicka wrote:
>
> > Hi,
> > this is minimal version of patch adding just the pointer compare.
> > Bootstrapped/regtested x86_64-linux, makes sense? :)
>
> Yes, obviously.
Thanks, i will go ahead with installing it.
Note that I have also tested removal of:
/* ??? Array types are not properly unified in all cases as we have
spurious changes in the index types for example. Removing this
causes all sorts of problems with the Fortran frontend. */
if (TREE_CODE (type1) == ARRAY_TYPE
&& TREE_CODE (type2) == ARRAY_TYPE)
return -1;
And it causes no regressions. I looked for the history and see you
added it in 2009 because Fortran mixes up array of chars with char
itself. I am not sure if that was fixed since then or it is just
about missing testcase?
It does not seem to be that important, but looks odd
and makes me woried about other changes :)
Honza
>
> Richard.
