From 52792faa0c8510b7bfc2c184c9f67d4a87f83215 Mon Sep 17 00:00:00 2001 From: Kamlesh Kumar Date: Tue, 3 Sep 2019 20:13:22 +0000 Subject: [PATCH] re PR tree-optimization/91504 (Inlining misses some logical operation folding) PR tree-optimization/91504 * match.pd: Add ((~a & b) ^a) --> (a | b). PR tree-optimization/91504 gcc.dg/tree-ssa/pr91504.c: New test. From-SVN: r275354 --- gcc/ChangeLog | 5 +++++ gcc/match.pd | 5 +++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.dg/tree-ssa/pr91504.c | 18 ++++++++++++++++++ 4 files changed, 33 insertions(+) create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr91504.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6ac810fca55..abbbf16781f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2019-09-03 Kamlesh Kumar + + PR tree-optimization/91504 + * match.pd: Add ((~a & b) ^a) --> (a | b). + 2019-09-03 Jakub Jelinek PR target/91604 diff --git a/gcc/match.pd b/gcc/match.pd index cd75dacd9cd..1d13543a615 100644 --- a/gcc/match.pd +++ b/gcc/match.pd @@ -831,6 +831,11 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT) (bit_xor:c (bit_and:cs @0 (bit_not @1)) (bit_not @0)) (bit_not (bit_and @0 @1))) +/* (~a & b) ^ a --> (a | b) */ +(simplify + (bit_xor:c (bit_and:cs (bit_not @0) @1) @0) + (bit_ior @0 @1)) + /* (a | b) & ~(a ^ b) --> a & b */ (simplify (bit_and:c (bit_ior @0 @1) (bit_not (bit_xor:c @0 @1))) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 44c43d26b93..4cf3e9b1ec4 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2019-09-03 Kamlesh Kumar + + PR tree-optimization/91504 + gcc.dg/tree-ssa/pr91504.c: New test. + 2019-09-03 Jakub Jelinek PR target/91604 diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr91504.c b/gcc/testsuite/gcc.dg/tree-ssa/pr91504.c new file mode 100644 index 00000000000..a52dea490e5 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr91504.c @@ -0,0 +1,18 @@ +/* { dg-do compile } */ +/* { dg-options "-O -fdump-tree-optimized-raw" } */ + +static inline unsigned deposit32(unsigned value, int start, int length, + unsigned fieldval) +{ + unsigned mask = (~0U >> (32 - length)) << start; + return (value & ~mask) | ((fieldval << start) & mask); +} + +unsigned foo(unsigned value) +{ + return deposit32(value, 10, 1, 1); +} + +/* { dg-final { scan-tree-dump-not "bit_and_expr" "optimized" } } */ +/* { dg-final { scan-tree-dump-not "bit_xor_expr" "optimized" } } */ +/* { dg-final { scan-tree-dump-not "bit_not_expr" "optimized" } } */ -- 2.30.2