diff options
author | Aldy Hernandez <aldyh@redhat.com> | 2021-06-14 12:51:48 +0200 |
---|---|---|
committer | Aldy Hernandez <aldyh@redhat.com> | 2021-06-15 18:41:09 +0200 |
commit | 4602714382f7a4603290b1ceb3121d9187debdb9 (patch) | |
tree | c6b6cc3441a3622800e7608583b9811690db7ed3 | |
parent | 913b13fcb1dceea0e57a04cb77b11097b132cbf6 (diff) | |
download | gcc-4602714382f7a4603290b1ceb3121d9187debdb9.zip gcc-4602714382f7a4603290b1ceb3121d9187debdb9.tar.gz gcc-4602714382f7a4603290b1ceb3121d9187debdb9.tar.bz2 |
range-ops: (nonzero | X) is nonzero
For bitwise or, nonzero|X is always nonzero. Make sure we don't drop to
varying in this case.
gcc/ChangeLog:
* range-op.cc (operator_bitwise_or::wi_fold): Make sure
nonzero|X is nonzero.
(range_op_bitwise_and_tests): Add tests for above.
-rw-r--r-- | gcc/range-op.cc | 31 |
1 files changed, 26 insertions, 5 deletions
diff --git a/gcc/range-op.cc b/gcc/range-op.cc index 742e546..e805f26 100644 --- a/gcc/range-op.cc +++ b/gcc/range-op.cc @@ -2534,11 +2534,20 @@ operator_bitwise_or::wi_fold (irange &r, tree type, new_lb = wi::max (new_lb, lh_lb, sign); if (wi::lt_p (rh_ub, 0, sign)) new_lb = wi::max (new_lb, rh_lb, sign); - // If the limits got swapped around, return varying. - if (wi::gt_p (new_lb, new_ub,sign)) - r.set_varying (type); - else - value_range_with_overflow (r, type, new_lb, new_ub); + // If the limits got swapped around, return a conservative range. + if (wi::gt_p (new_lb, new_ub, sign)) + { + // Make sure that nonzero|X is nonzero. + if (wi::gt_p (lh_lb, 0, sign) + || wi::gt_p (rh_lb, 0, sign) + || wi::lt_p (lh_ub, 0, sign) + || wi::lt_p (rh_ub, 0, sign)) + r.set_nonzero (type); + else + r.set_varying (type); + return; + } + value_range_with_overflow (r, type, new_lb, new_ub); } bool @@ -3744,6 +3753,18 @@ range_op_bitwise_and_tests () i1 = int_range<1> (integer_type_node); op_bitwise_and.op1_range (res, integer_type_node, i1, i2); ASSERT_TRUE (res == int_range<1> (integer_type_node)); + + // (NONZERO | X) is nonzero. + i1.set_nonzero (integer_type_node); + i2.set_varying (integer_type_node); + op_bitwise_or.fold_range (res, integer_type_node, i1, i2); + ASSERT_TRUE (res.nonzero_p ()); + + // (NEGATIVE | X) is nonzero. + i1 = int_range<1> (INT (-5), INT (-3)); + i2.set_varying (integer_type_node); + op_bitwise_or.fold_range (res, integer_type_node, i1, i2); + ASSERT_FALSE (res.contains_p (INT (0))); } void |