And in the rl78 port.  Tested by building the rl78 targets in config-all.mk.

Installed on the trunk.

Jeff
commit 6d8cde85a30e36e5b5842b8d66837a8b4815d197
Author: Jeff Law <l...@tor.usersys.redhat.com>
Date:   Mon Sep 28 19:25:04 2015 -0400

    [PATCH] Fix undefined behaviour in rl78 port
            * config/rl78/rl78-expand.md (movqi): Fix undefined left shift
            behaviour.

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1b9985a..79dc89f 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,8 @@
 2015-09-29  Jeff Law  <l...@redhat.com>
 
+       * config/rl78/rl78-expand.md (movqi): Fix undefined left shift
+       behaviour.
+
        * config/msp430/msp430.c (msp430_legitimate_constant): Fix undefined
        left shift behaviour.
        * config/msp430/constraints.md ('L' constraint): Similarly.
diff --git a/gcc/config/rl78/rl78-expand.md b/gcc/config/rl78/rl78-expand.md
index 0335a4d..67e6620 100644
--- a/gcc/config/rl78/rl78-expand.md
+++ b/gcc/config/rl78/rl78-expand.md
@@ -48,7 +48,7 @@
        && ! REG_P (operands[0]))
        operands[1] = copy_to_mode_reg (QImode, operands[1]);
 
-    if (CONST_INT_P (operands[1]) && ! IN_RANGE (INTVAL (operands[1]), (-1 << 
8) + 1, (1 << 8) - 1))
+    if (CONST_INT_P (operands[1]) && ! IN_RANGE (INTVAL (operands[1]), 
(HOST_WIDE_INT_M1U << 8) + 1, (1 << 8) - 1))
       FAIL;
   }
 )

Reply via email to