On Mon, 29 May 2017, Yuri Gribov wrote:

> This no longer fixes the PR but still works in some cases as
> demonstrated by the test. So I decided to keep it.

As Richard noticed you don't need widest_ints but can use wide_ints.
Please use == 0 instead of ! on wide-ints as well.

+(for cmp (le gt)
+ (simplify
..
+      (switch
+       (if (cmp == LE_EXPR)
+       (eq:type (bit_and @1 { wide_int_to_tree (ty, hi_bits); }) { 
build_zero_cst (ty); }))
+       (if (cmp == GT_EXPR)
+       (ne:type (bit_and @1 { wide_int_to_tree (ty, hi_bits); }) { 
build_zero_cst (ty); })))))))))

long lines plus you can simplify this with using

 (for cmp (le gt)
      eqcmp (eq ne)
 ...

     (eqcmp (bit_and @1 { wide_int_to_tree (ty, hi_bits); }) 
{build_zero_cst (ty); }))))

no need to spell out :type on the result as well.

Richard.

Reply via email to