>I think this is a fairly reasonable minimal fix. For 4.8 we could >experiment whether to always do this, regardless of s_r_c_f_m_p. > >Ok if bootstrapped and tested on a primary target (i.e. linux) and >tested on Blackfin. > > >Bernd
Thanks again, Bernd.
Forwarding to gcc-patches to give people a couple of days to object.
Tested on linux and bfin.
Stu
diff --git a/gcc/ifcvt.c b/gcc/ifcvt.c
index 8d81c89..e4e13ab 100644
--- a/gcc/ifcvt.c
+++ b/gcc/ifcvt.c
@@ -2295,7 +2295,9 @@ noce_get_condition (rtx jump, rtx *earliest, bool
then_else_reversed)
cond = XEXP (SET_SRC (set), 0);
tmp = XEXP (cond, 0);
- if (REG_P (tmp) && GET_MODE_CLASS (GET_MODE (tmp)) == MODE_INT)
+ if (REG_P (tmp) && GET_MODE_CLASS (GET_MODE (tmp)) == MODE_INT
+ && (GET_MODE (tmp) != BImode
+ || !targetm.small_register_classes_for_mode_p (BImode)))
{
*earliest = jump;
upstream2.patch
Description: upstream2.patch
