Skip to content

R
riscv-gcc-1
Commit e5075818 by Richard Kenner
Options

(fold, case EQ_EXPR): Remove code that converts mod to unsigned mod; 

it isn't correct for negative operands.

From-SVN: r4995
parent 437b3c09
Showing with 5 additions and 0 deletions
gcc/fold-const.c
...@@ -4203,6 +4203,10 @@ fold (expr) ...@@ -4203,6 +4203,10 @@ fold (expr)
arg1)); arg1));
} }
/* It would be nice to do this since it generates better code.
Unfortunately, it doesn't produce the correct result if the
first operand is negative. */
#if 0
/* If this is an NE or EQ comparison of zero against the result of a /* 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 signed MOD operation, make the MOD operation unsigned since it
is simpler and equivalent. */ is simpler and equivalent. */
...@@ -4221,6 +4225,7 @@ fold (expr) ...@@ -4221,6 +4225,7 @@ fold (expr)
return build (code, type, newmod, convert (newtype, arg1)); return build (code, type, newmod, convert (newtype, arg1));
} }
#endif
/* 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. */
......
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