From 2066ef6ac8cd3e04f46b9fceeb7b8d16eff62cf3 Mon Sep 17 00:00:00 2001 From: Prathamesh Kulkarni Date: Thu, 20 Oct 2016 07:55:28 +0000 Subject: [PATCH] re PR tree-optimization/53979 (((a ^ b) | a) not optimized to (a | b)) 2016-10-20 Prathamesh Kulkarni PR tree-optimization/53979 * match.pd ((a ^ b) | a -> a | b): New pattern. testsuite/ * gcc.dg/pr53979-1.c: New test-case. * gcc.dg/pr53979-2.c: Likewise. From-SVN: r241360 --- gcc/ChangeLog | 5 +++++ gcc/match.pd | 6 ++++++ gcc/testsuite/ChangeLog | 6 ++++++ gcc/testsuite/gcc.dg/pr53979-1.c | 9 +++++++++ gcc/testsuite/gcc.dg/pr53979-2.c | 11 +++++++++++ 5 files changed, 37 insertions(+) create mode 100644 gcc/testsuite/gcc.dg/pr53979-1.c create mode 100644 gcc/testsuite/gcc.dg/pr53979-2.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2b26a38ba9e..d4779690e9e 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2016-10-20 Prathamesh Kulkarni + + PR tree-optimization/53979 + * match.pd ((a ^ b) | a -> a | b): New pattern. + 2016-10-19 John David Anglin * config/pa/pa64-hpux.h (PA_INIT_FRAME_DUMMY_ASM_OP): Move to diff --git a/gcc/match.pd b/gcc/match.pd index b782a1ed2f4..78884591c8f 100644 --- a/gcc/match.pd +++ b/gcc/match.pd @@ -541,6 +541,12 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT) (bit_ior:c (bit_and @0 INTEGER_CST@2) (bit_and (bit_not @0) INTEGER_CST@1)) (if (wi::bit_not (@2) == @1) (bit_xor @0 @1))) + +/* PR53979: Transform ((a ^ b) | a) -> (a | b) */ +(simplify + (bit_ior:c (bit_xor:c @0 @1) @0) + (bit_ior @0 @1)) + /* Simplify (~X & Y) to X ^ Y if we know that (X & ~Y) is 0. */ #if GIMPLE (simplify diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8f5230f515a..152b1b26bba 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2016-10-20 Prathamesh Kulkarni + + PR tree-optimization/53979 + * gcc.dg/pr53979-1.c: New test-case. + * gcc.dg/pr53979-2.c: Likewise. + 2016-10-19 Bernd Edlinger * c-c++-common/Wint-in-bool-context-2.c: New test. diff --git a/gcc/testsuite/gcc.dg/pr53979-1.c b/gcc/testsuite/gcc.dg/pr53979-1.c new file mode 100644 index 00000000000..aee54f52378 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr53979-1.c @@ -0,0 +1,9 @@ +/* { dg-do compile } */ +/* { dg-options "-fdump-tree-gimple" } */ + +unsigned f1(unsigned a, unsigned b) +{ + return (a ^ b) | a; +} + +/* { dg-final { scan-tree-dump "a | b" "gimple" } } */ diff --git a/gcc/testsuite/gcc.dg/pr53979-2.c b/gcc/testsuite/gcc.dg/pr53979-2.c new file mode 100644 index 00000000000..be2607aea39 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr53979-2.c @@ -0,0 +1,11 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -fdump-tree-forwprop-details" } */ + +unsigned f(unsigned a, unsigned b) +{ + unsigned t1 = a ^ b; + unsigned t2 = t1 | a; + return t2; +} + +/* { dg-final { scan-tree-dump "gimple_simplified to t2_\[0-9\] = a_\[0-9\]*\\(D\\) | b_\[0-9\]*\\(D\\)" "forwprop1" } } */ -- 2.30.2