This patch introduces a simplification rule in match.pd to reduce bitwise
expressions against zero. Specifically, it simplifies patterns where a
variable checked against zero is combined via bitwise AND/OR with a compounded
bitwise OR check against zero.

        PR tree-optimization/125442

gcc/ChangeLog:

        * match.pd: Add simplification rules for
        (a == 0) | ((a | b) == 0) -> (a == 0) and
        (a != 0) & ((a | b) != 0) -> (a != 0).

gcc/testsuite/ChangeLog:

        * gcc.dg/int-bwise-opt-3.c: New test.
        * gcc.dg/int-bwise-opt-4.c: New test.

Suggested-by: Andrew Pinski <[email protected]>
Signed-off-by: Souradipto Das <[email protected]>
---
Changes in v3:
 - Squashed the match.pd optimization rules and testcases into a single unified 
patch.
Changes in v2:
 - Added dedicated testcases for the new bitwise optimization rules as 
suggested by Andrew Pinski.
 gcc/match.pd                           | 9 +++++++++
 gcc/testsuite/gcc.dg/int-bwise-opt-3.c | 9 +++++++++
 gcc/testsuite/gcc.dg/int-bwise-opt-4.c | 9 +++++++++
 3 files changed, 27 insertions(+)
 create mode 100644 gcc/testsuite/gcc.dg/int-bwise-opt-3.c
 create mode 100644 gcc/testsuite/gcc.dg/int-bwise-opt-4.c

diff --git a/gcc/match.pd b/gcc/match.pd
index 8a2de136e..4a2f9a784 100644
--- a/gcc/match.pd
+++ b/gcc/match.pd
@@ -6660,6 +6660,15 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT)
  (simplify
   (bitop (neeql @0 @1) (neeqr (bit_ior @0 @1) integer_zerop))
   { constant_boolean_node (bitop == BIT_IOR_EXPR, type); }))
+
+/* (a == 0) | ((a | b) == 0) -> (a == 0) -- PR125442
+   (a != 0) & ((a | b) != 0) -> (a != 0) -- PR125442 */
+
+(for bitop (bit_and bit_ior)
+     neeq  (ne      eq)
+(simplify
+  (bitop:c (neeq@2 @0 integer_zerop) (neeq (bit_ior:c @0 @1) integer_zerop))
+  @2))
 #endif
 
 /* These was part of minmax phiopt.  */
diff --git a/gcc/testsuite/gcc.dg/int-bwise-opt-3.c 
b/gcc/testsuite/gcc.dg/int-bwise-opt-3.c
new file mode 100644
index 000000000..199e85dbe
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/int-bwise-opt-3.c
@@ -0,0 +1,9 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -fdump-tree-optimized" } */
+
+int f1(int a, int b)
+{
+  return (a == 0) | ((a | b) == 0);
+}
+
+/* { dg-final { scan-tree-dump-times "\\\|" 0 "optimized" } } */
diff --git a/gcc/testsuite/gcc.dg/int-bwise-opt-4.c 
b/gcc/testsuite/gcc.dg/int-bwise-opt-4.c
new file mode 100644
index 000000000..6989f71b8
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/int-bwise-opt-4.c
@@ -0,0 +1,9 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -fdump-tree-optimized" } */
+
+int f2(int a, int b)
+{
+  return (a != 0) & ((a | b) != 0);
+}
+
+/* { dg-final { scan-tree-dump-times "\&" 0 "optimized" } } */
-- 
2.43.0

Reply via email to