Hi Peter, on 2024/2/20 06:35, Peter Bergner wrote: > rs6000: Update instruction counts due to combine changes [PR112103] > > The PR91865 combine fix changed instruction counts slightly for rlwinm-0.c. > Adjust expected instruction counts accordingly. > > This passed on both powerpc64le-linux and powerpc64-linux running the > testsuite in both 32-bit and 64-bit modes. Ok for trunk?
OK for trunk, thanks for fixing! > > FYI, I will open a new bug to track the removing of the superfluous > insns detected in PR112103. Hope this test case will become not fragile any more once this filed issue gets fixed. :) BR, Kewen > > > Peter > > > gcc/testsuite/ > PR target/112103 > * gcc.target/powerpc/rlwinm-0.c: Adjust expected instruction counts. > > diff --git a/gcc/testsuite/gcc.target/powerpc/rlwinm-0.c > b/gcc/testsuite/gcc.target/powerpc/rlwinm-0.c > index 4f4fca2d8ef..a10d9174306 100644 > --- a/gcc/testsuite/gcc.target/powerpc/rlwinm-0.c > +++ b/gcc/testsuite/gcc.target/powerpc/rlwinm-0.c > @@ -4,10 +4,10 @@ > /* { dg-final { scan-assembler-times {(?n)^\s+[a-z]} 6739 { target ilp32 } } > } */ > /* { dg-final { scan-assembler-times {(?n)^\s+[a-z]} 9716 { target lp64 } } > } */ > /* { dg-final { scan-assembler-times {(?n)^\s+blr} 3375 } } */ > -/* { dg-final { scan-assembler-times {(?n)^\s+rldicl} 3081 { target lp64 } } > } */ > +/* { dg-final { scan-assembler-times {(?n)^\s+rldicl} 3090 { target lp64 } } > } */ > > /* { dg-final { scan-assembler-times {(?n)^\s+rlwinm} 3197 { target ilp32 } > } } */ > -/* { dg-final { scan-assembler-times {(?n)^\s+rlwinm} 3093 { target lp64 } } > } */ > +/* { dg-final { scan-assembler-times {(?n)^\s+rlwinm} 3084 { target lp64 } } > } */ > /* { dg-final { scan-assembler-times {(?n)^\s+rotlwi} 154 } } */ > /* { dg-final { scan-assembler-times {(?n)^\s+srwi} 13 { target ilp32 } } } > */ > /* { dg-final { scan-assembler-times {(?n)^\s+srdi} 13 { target lp64 } } } */