aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAldy Hernandez <aldyh@redhat.com>2023-07-06 10:55:46 +0200
committerAldy Hernandez <aldyh@redhat.com>2023-07-12 23:15:52 +0200
commit7a5e47658904de9dc90f1292f3f69769177706f9 (patch)
treee13055c58f64a4010af7ea27d710decabb942f17
parent137fb7077f7711e2e09ee8f82650fe7d93de6a4d (diff)
downloadgcc-7a5e47658904de9dc90f1292f3f69769177706f9.zip
gcc-7a5e47658904de9dc90f1292f3f69769177706f9.tar.gz
gcc-7a5e47658904de9dc90f1292f3f69769177706f9.tar.bz2
[range-op] Take known mask into account for bitwise ands [PR107043]
PR tree-optimization/107043 gcc/ChangeLog: * range-op.cc (operator_bitwise_and::op1_range): Update bitmask. gcc/testsuite/ChangeLog: * gcc.dg/tree-ssa/pr107043.c: New test.
-rw-r--r--gcc/range-op.cc8
-rw-r--r--gcc/testsuite/gcc.dg/tree-ssa/pr107043.c22
2 files changed, 30 insertions, 0 deletions
diff --git a/gcc/range-op.cc b/gcc/range-op.cc
index 56e80c9..6b5d4f2 100644
--- a/gcc/range-op.cc
+++ b/gcc/range-op.cc
@@ -3463,6 +3463,14 @@ operator_bitwise_and::op1_range (irange &r, tree type,
if (r.undefined_p ())
set_nonzero_range_from_mask (r, type, lhs);
+ // For MASK == op1 & MASK, all the bits in MASK must be set in op1.
+ wide_int mask;
+ if (lhs == op2 && lhs.singleton_p (mask))
+ {
+ r.update_bitmask (irange_bitmask (mask, ~mask));
+ return true;
+ }
+
// For 0 = op1 & MASK, op1 is ~MASK.
if (lhs.zero_p () && op2.singleton_p ())
{
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr107043.c b/gcc/testsuite/gcc.dg/tree-ssa/pr107043.c
new file mode 100644
index 0000000..af5df22
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/tree-ssa/pr107043.c
@@ -0,0 +1,22 @@
+// { dg-do compile }
+// { dg-options "-O2 -fdump-tree-evrp" }
+
+int g0(int n)
+{
+ int n1 = n & 0x8000;
+ if (n1 == 0)
+ return 1;
+ // n1 will be 0x8000 here.
+ return (n1 >> 15) & 0x1;
+}
+
+int g1(int n)
+{
+ int n1 = n & 0x8000;
+ if (n1 == 0)
+ return 1;
+ // n>>15 will be xxxxxx1 here.
+ return (n >> 15) & 0x1;
+}
+
+// { dg-final { scan-tree-dump-times "return 1;" 2 "evrp" } }