> Please find attached the updated patch. Thanks. Still, although reg_mode[regno] is indeed tested above, in
+ for (i = hard_regno_nregs[regno][mode] - 1; i > 0; i--) + if (reg_mode[i] != BLKmode) + return false; this should be reg_mode[regno + i] instead of reg_mode[i]. And one the nice benefits of the switch to C++ is that you can now declare the iteration variable within the 'for' construct. So, if you aren't planning to backport the patch to branches prior to 4.8, you can use the idiom in the newmove2add_record_mode and move2add_valid_value_p functions. The patch is OK with these modifications if it still passes testing. -- Eric Botcazou
