[AArch64] PR target/20666, fix wrong encoding of new introduced BFC pseudo
Jan Beulich
JBeulich@suse.com
Mon Nov 7 11:55:00 GMT 2016
>>> On 10.10.16 at 18:40, <jiong.wang@foss.arm.com> wrote:
> On 24/11/15 11:17, Matthew Wahab wrote:
>> opcodes/
>> 2015-11-24 Matthew Wahab <matthew.wahab@arm.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
> target/20666.
>
> +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.
Jan
More information about the Binutils
mailing list