On Tue, May 24, 2016 at 04:55:45PM -0500, Pat Haugen wrote:
> The following simplifies the given testcase so it is no longer sensitive to
> subreg (and hopefully other) codegen changes. Tested on powerpc64, ok for
> trunk?
>
> -Pat
>
>
> testsuite/ChangeLog:
> 2016-05-24 Pat Haugen <pthau...@us.ibm.com>
>
> PR target/71050
> * gcc.target/powerpc/lhs-1.c: Fix testcase to avoid subreg changes.
It is okay for trunk. One thing (well, two)...
> Index: gcc/testsuite/gcc.target/powerpc/lhs-1.c
> ===================================================================
> --- gcc/testsuite/gcc.target/powerpc/lhs-1.c (revision 236325)
> +++ gcc/testsuite/gcc.target/powerpc/lhs-1.c (working copy)
> @@ -4,19 +4,12 @@
> /* { dg-options "-O2 -mcpu=power5" } */
> /* { dg-final { scan-assembler-times "nop" 3 } } */
>
> -/* Test generation of nops in load hit store situation. */
> +/* Test generation of nops in load hit store situation. Make sure enough nop
> insns are
> + generated to move the load to a new dispatch group. With the simple
> stw/lwz pair below,
> + that would be 3 nop insns for Power5. */
Long lines; dot space space.
Thanks,
Segher
