And now the SH port in the obvious way. Verified by building all the sh configurations in config-list.mk with a trunk compiler.
Installed on the trunk, Jeff
commit ac55233b083fc30445141c530fcf472786f935e7 Author: Jeff Law <l...@tor.usersys.redhat.com> Date: Sat Sep 26 03:24:07 2015 -0400 [PATCH] Fix undefined behaviour in SH port * config/sh/sh.h (CONST_OK_FOR_J16): Fix left shift undefined behaviour diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6887b14..28c6bf7 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,8 @@ 2015-09-26 Jeff Law <l...@redhat.com> + * config/sh/sh.h (CONST_OK_FOR_J16): Fix left shift undefined + behaviour + * config/mips/mips.c (mips_compute_frame_info): Fix left shift undefined behaviour. diff --git a/gcc/config/sh/sh.h b/gcc/config/sh/sh.h index 7aa489d..b18cc1f 100644 --- a/gcc/config/sh/sh.h +++ b/gcc/config/sh/sh.h @@ -1176,7 +1176,7 @@ extern enum reg_class regno_reg_class[FIRST_PSEUDO_REGISTER]; #define CONST_OK_FOR_J16(VALUE) \ ((HOST_BITS_PER_WIDE_INT >= 64 && (VALUE) == (HOST_WIDE_INT) 0xffffffff) \ - || (HOST_BITS_PER_WIDE_INT >= 64 && (VALUE) == (HOST_WIDE_INT) -1 << 32)) + || (HOST_BITS_PER_WIDE_INT >= 64 && (VALUE) == (HOST_WIDE_INT) (HOST_WIDE_INT_M1U << 32))) #define CONST_OK_FOR_K08(VALUE) (((HOST_WIDE_INT)(VALUE))>= 0 \ && ((HOST_WIDE_INT)(VALUE)) <= 255)