[PATCH], V7, #2 of 7, Use PLI to load up 32-bit SImode constants

Segher Boessenkool segher@kernel.crashing.org
Tue Nov 26 00:55:00 GMT 2019


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



More information about the Gcc-patches mailing list