[PATCH] Fix build breakage on GNU/Linux AArch64 and use std::vector on tdesc.reg_defs
Sergio Durigan Junior
sergiodj@redhat.com
Sun Sep 10 16:12:00 GMT 2017
On Sunday, September 10 2017, Simon Marchi wrote:
> On 2017-09-10 07:50, Sergio Durigan Junior wrote:
>> This patch fixes the build breakage that has been happening on AArch64
>> since September 5th. The breakage was introduced by the following
>> commit:
>>
>> author Yao Qi <yao.qi@linaro.org>
>> Tue, 5 Sep 2017 04:54:52 -0400 (09:54 +0100)
>> committer Yao Qi <yao.qi@linaro.org>
>> Tue, 5 Sep 2017 04:54:52 -0400 (09:54 +0100)
>> commit f7000548a2b79d7e5cb924468117ca4245e6b820
>>
>> Use VEC for target_desc.reg_defs
>>
>> The build log for this commit can be seen here:
>>
>>
>> <https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2696/steps/compile%20gdb/logs/stdio>
>>
>> And the underlying problem is that the code is not calling the new
>> function "allocate_target_description" to allocate the "struct
>> target_desc" using "new" instead of XNEW, which end up not properly
>> initializing the fields of the structure.
>>
>> I took the opportunity to convert the code to std::vector (instead of
>> VEC), which makes things even simpler. This has been regtested on the
>> BuildBot, without regressions, and fixes the build breakage.
>
> Hi Sergio,
>
> This patch seems to do two independent changes:
>
> - use allocate_target_description instead of XNEW
> - replace VEC with std::vector
>
> Can you post separate patches? You can push the first one directly, I
> would call it obvious (and it's the important bits to fix the build).
> The second one, Yao might want to take a look.
Sure thing, I'll split it and resubmit. Thanks.
--
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/
More information about the Gdb-patches
mailing list