[PATCH 2/5] PowerPC D-form prefixed loads and stores

[Alan Modra]

On Thu, Jun 20, 2019 at 06:04:43AM -0600, Jan Beulich wrote:
> >>> Alan Modra <amodra@gmail.com> 05/24/19 3:34 AM >>>
> >@@ -7815,6 +7969,32 @@ const unsigned int powerpc_num_opcodes =
>  >
> >const struct powerpc_opcode prefix_opcodes[] = {
> >{"pnop",	  PMRR,		       PREFIX_MASK,	POWERXX, 0,	{0}},
> >+{"pli",		  PMLS|OP(14),	       P_DRAPCREL_MASK,	POWERXX, 0,	{RT, SI34}},
> >+{"paddi",	  PMLS|OP(14),	       P_D_MASK,	POWERXX, 0,	{RT, RA0, SI34, PCREL0}},
> >+{"psubi",	  PMLS|OP(14),	       P_D_MASK,	POWERXX, 0,	{RT, RA0, NSI34, PCREL0}},
> >+{"pla",		  PMLS|OP(14),	       P_D_MASK,	POWERXX, 0,	{RT, D34, PRA0, PCREL}},
> >+{"plwz",	  PMLS|OP(32),	       P_D_MASK,	POWERXX, 0,	{RT, D34, PRA0, PCREL}},
> >+{"plbz",	  PMLS|OP(34),	       P_D_MASK,	POWERXX, 0,	{RT, D34, PRA0, PCREL}},
> >+{"pstw",	  PMLS|OP(36),	       P_D_MASK,	POWERXX, 0,	{RS, D34, PRA0, PCREL}},
> >+{"pstb",	  PMLS|OP(38),	       P_D_MASK,	POWERXX, 0,	{RS, D34, PRA0, PCREL}},
> >+{"plhz",	  PMLS|OP(40),	       P_D_MASK,	POWERXX, 0,	{RT, D34, PRA0, PCREL}},
> >+{"plwa",	  P8LS|OP(41),	       P_D_MASK,	POWERXX, 0,	{RT, D34, PRA0, PCREL}},
> 
> Comparing with the neighboring ones - isn't this supposed to use PMLS?

No.  The PMLS forms use the same opcode in the suffix word.  plwa
doesn't share the lwa encoding, and doesn't have the lwa restriction
of the offset being a multiple of 4.

-- 
Alan Modra
Australia Development Lab, IBM