Hi! On Fri, Apr 13, 2018 at 04:43:02PM -0400, Vladimir Makarov wrote: > On 04/13/2018 03:58 PM, Alexander Monakov wrote: > >Here's another compact variant: > > > > regno = reg_renumber[regno]; > > if (regno < 0) > > regno = cl == NO_REGS ? -1 : ira_class_hard_regs[cl][1]; > > > Thanks, Alexander. I like your variant more.
Me too :-) > --- testsuite/gcc.target/powerpc/pr79916.c (nonexistent) > +++ testsuite/gcc.target/powerpc/pr79916.c (working copy) > @@ -0,0 +1,556 @@ > +/* { dg-do compile { target { powerpc64le-*-* } } } */ Is the testcase specific to LE? If not, please use powerpc*-*-*, or powerpc*-*-* && lp64 if it needs 64-bit. > +/* { dg-options "-Idfp -fno-expensive-optimizations --param > ira-max-conflict-table-size=0 -mno-popcntd -O3" } */ I think you can drop the -Idfp ? If that option would do anything we have bigger problems ;-) Looks fine to me otherwise. Thanks for working on this! Segher