This is the mail archive of the
mailing list for the binutils project.
Re: [AArch64] PR target/20666, fix wrong encoding of new introduced BFC pseudo
- From: Jiong Wang <jiong dot wang at foss dot arm dot com>
- To: Jan Beulich <JBeulich at suse dot com>
- Cc: "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Mon, 7 Nov 2016 12:22:04 +0000
- Subject: Re: [AArch64] PR target/20666, fix wrong encoding of new introduced BFC pseudo
- Authentication-results: sourceware.org; auth=none
- References: <firstname.lastname@example.org> <email@example.com> <582079AA020000780011CA44@prv-mh.provo.novell.com>
On 07/11/16 11:55, Jan Beulich wrote:
On 10.10.16 at 18:40, <firstname.lastname@example.org> wrote:
On 24/11/15 11:17, Matthew Wahab wrote:
2015-11-24 Matthew Wahab <email@example.com>
* aarch64-asm-2.c: Regenerate.
* aarch64-asm.c (convert_bfc_to_bfm): New.
There looks like a copy & paste error in this function, and caused PR
+convert_bfc_to_bfm (aarch64_inst *inst)
+ int64_t lsb, width;
+ /* Insert XZR. */
+ copy_operand_info (inst, 3, 2);
+ copy_operand_info (inst, 2, 1);
+ copy_operand_info (inst, 2, 0);
^~~~~~~ should be "0".
Why 0? That makes this particular copy_operand_info() invocation
a no-op afaict. Aiui you mean 1 here instead.
I mean 0 there. as "BFC Xd, lsb, width" is the simplified form of
"BFI Xd, XZR, lsb, width".
Operands 1 is forced to XZR by the following code:
inst->operands.reg.regno = 0x1f;
But you are correct there, copy_operand_info (inst, 0, 0) then
we can delete that I think.