[PATCH] [AArch64] Fix erroneous use of spu architecture bfd

Luis Machado luis.machado@linaro.org
Sun Jan 5 12:00:00 GMT 2020


On 1/5/20 3:27 AM, Joel Brobecker wrote:
> Hi Luis,
> 
>> While investigating some SVE code, i noticed the use of two spu bfd variables.
>>
>> This looks like an oversight, as the "id" field is available for non-spu
>> architectures as well, even though its primary use was the Cell BE
>> architecture.
>>
>> The following patch fixes this.
>>
>> gdb/ChangeLog:
>>
>> 2020-01-03  Luis Machado  <luis.machado@linaro.org>
>>
>> 	* aarch64-linux-nat.c
>> 	(aarch64_linux_nat_target::thread_architecture): Use bfd_arch_aarch64
>> 	and bfd_mach_aarch64.
>>
>> Change-Id: I817673e9a12cadfa88638870f0ff524241cf6a34
> 
> 
> I think the patch looks pretty clear to me ;-), so you can go ahead
> and push.
> 
> Or perhaps, before doing so: Since the Gerrit experiment is being
> put on hold, you might want to remove your commit hook, and then
> remove the "Change-Id" field above...

Done now.

> 
> Thanks for the patch >

Pushed to master. Thanks!



More information about the Gdb-patches mailing list