Commit 05a0d5ea by Richard Kenner

(fold, case EQ_EXPR, LE_EXPR): If comparing results of signed MOD with

zero, use an unsigned MOD.

From-SVN: r4852
parent 40083ddf
...@@ -4163,6 +4163,25 @@ fold (expr) ...@@ -4163,6 +4163,25 @@ fold (expr)
arg1)); arg1));
} }
/* If this is an NE or EQ comparison of zero against the result of a
signed MOD operation, make the MOD operation unsigned since it
is simpler and equivalent. */
if ((code == NE_EXPR || code == EQ_EXPR)
&& integer_zerop (arg1)
&& ! TREE_UNSIGNED (TREE_TYPE (arg0))
&& (TREE_CODE (arg0) == TRUNC_MOD_EXPR
|| TREE_CODE (arg0) == CEIL_MOD_EXPR
|| TREE_CODE (arg0) == FLOOR_MOD_EXPR
|| TREE_CODE (arg0) == ROUND_MOD_EXPR))
{
tree newtype = unsigned_type (TREE_TYPE (arg0));
tree newmod = build (TREE_CODE (arg0), newtype,
convert (newtype, TREE_OPERAND (arg0, 0)),
convert (newtype, TREE_OPERAND (arg0, 1)));
return build (code, type, newmod, convert (newtype, arg1));
}
/* If this is an NE comparison of zero with an AND of one, remove the /* If this is an NE comparison of zero with an AND of one, remove the
comparison since the AND will give the correct value. */ comparison since the AND will give the correct value. */
if (code == NE_EXPR && integer_zerop (arg1) if (code == NE_EXPR && integer_zerop (arg1)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment