On Mon, Nov 25, 2019 at 05:17:08PM -0500, Michael Meissner wrote: > On Fri, Nov 22, 2019 at 06:20:52PM -0600, Segher Boessenkool wrote: > > > (define_split > > > [(set (match_operand:SI 0 "gpc_reg_operand") > > > (match_operand:SI 1 "const_int_operand"))] > > > "(unsigned HOST_WIDE_INT) (INTVAL (operands[1]) + 0x8000) >= 0x10000 > > > - && (INTVAL (operands[1]) & 0xffff) != 0" > > > + && (INTVAL (operands[1]) & 0xffff) != 0 && !TARGET_PREFIXED_ADDR" > > > [(set (match_dup 0) > > > (match_dup 2)) > > > (set (match_dup 0) > > > > Please use num_insns_constant, instead (and fix num_insns_constant_gpr > > so it knows about SIGNED_34BIT). > > The previous patch V6 #1 already had the modification for > num_insns_constant_gpr.
It's not in trunk yet? Segher