This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 2/5] PowerPC D-form prefixed loads and stores
- From: Alan Modra <amodra at gmail dot com>
- To: Jan Beulich <jbeulich at suse dot com>
- Cc: binutils at sourceware dot org
- Date: Thu, 20 Jun 2019 22:47:27 +0930
- Subject: Re: [PATCH 2/5] PowerPC D-form prefixed loads and stores
- References: <20190524013110.GC6820@bubble.grove.modra.org> <20190524013424.GE6820@bubble.grove.modra.org> <5D0B765B020000780015CF2D@prv1-mh.provo.novell.com>
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