On Mon, Dec 21, 2015 at 01:13:28PM +0000, Alan Lawrence wrote:
> ...the test passes with --param sra-max-scalarization-size-Ospeed.
>
> Verified on aarch64 and with stage1 compiler for hppa, powerpc, sparc, s390.
How did you test this on s390? For me, the test still fails
unless I add -march=z13 (s390x).
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/tree-ssa/ssa-dom-cse-2.c: Remove XFAIL for powerpc(32), hppa,
> aarch64, sparc, s390. Add --param sra-max-scalarization-size-Ospeed.
> ---
> gcc/testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c
> b/gcc/testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c
> index 9eccdc9..748448e 100644
> --- a/gcc/testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c
> +++ b/gcc/testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c
> @@ -1,5 +1,5 @@
> /* { dg-do compile } */
> -/* { dg-options "-O3 -fno-tree-fre -fno-tree-pre -fdump-tree-optimized" } */
> +/* { dg-options "-O3 -fno-tree-fre -fno-tree-pre -fdump-tree-optimized
> --param sra-max-scalarization-size-Ospeed=32" } */
>
> int
> foo ()
> @@ -17,7 +17,8 @@ foo ()
> /* After late unrolling the above loop completely DOM should be
> able to optimize this to return 28. */
>
> -/* See PR63679 and PR64159, if the target forces the initializer to memory
> then
> - DOM is not able to perform this optimization. */
> +/* On alpha, the vectorizer generates writes of two vector elements at once,
> + but the loop reads only one element at a time, and DOM cannot resolve
> these.
> + The same happens on powerpc depending on the SIMD support available. */
>
> -/* { dg-final { scan-tree-dump "return 28;" "optimized" { xfail aarch64*-*-*
> alpha*-*-* hppa*-*-* powerpc*-*-* sparc*-*-* s390*-*-* } } } */
> +/* { dg-final { scan-tree-dump "return 28;" "optimized" { xfail alpha*-*-*
> powerpc64*-*-* } } } */
> --
> 1.9.1
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
