Hi!
On Wed, Oct 21, 2020 at 01:00:20PM -0500, Peter Bergner wrote:
> PR97360 shows a problem in how we create our PXI and POI modes that cause
> an ICE in the ranger pass. The problem seems to be that the extra call
> to build_distinct_type_copy() also creates new TYPE_{MIN,MAX}_VALUEs that
> are not compatible/the same as the base type itself. The simple "fix" is
> to actually remove the unneeded build_distinct_type_copy(), since according
> to richi, the types returned from make_unsigned_type() are already distinct.
>
> The following patch from Andrew and richi fixes the ICE on Martin's test
> case and passes bootstrap and regtesting on powerpc64le-linux.
> Ok for trunk?
Yes, okay for trunk. Thanks!
> Since the ranger code that triggered this doesn't seem to be in GCC 10,
> I assume we do not want to backport this this change?
No, please do, in a week or so, it is a pretty serious problem that we
could just asa well run into some other way, as far as I can see?
Segher
> gcc/
> PR target/97360
> * config/rs6000/rs6000-call.c (rs6000_init_builtins): Remove call to
> build_distinct_type_copy().
>
> gcc/testsuite/
> PR target/97360
> * gcc.target/powerpc/pr97360.c: New test.
