[PATCH 2/2] D: Sync libiberty with upstream GCC.

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH 2/2] D: Sync libiberty with upstream GCC.

Iain Buclaw
Hi,

This patch synchronizes GDB with GCC's copy of libiberty - the D
demangling part.

Iain.

---

02-sync-libiberty-d.patch (47K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 2/2] D: Sync libiberty with upstream GCC.

Simon Marchi
On 2017-06-06 22:36, Iain Buclaw wrote:
> Hi,
>
> This patch synchronizes GDB with GCC's copy of libiberty - the D
> demangling part.
>
> Iain.
>
> ---

Hi Iain,

Starting with this commit, I see this failure in the D tests:

FAIL: gdb.dlang/demangle.exp: _D8demangle4test6__initZ

This command

(gdb) demangle _D8demangle4test6__initZ
demangle.test.init$

now gives this result

(gdb) demangle _D8demangle4test6__initZ
initializer for demangle.test

Is it simply that the test needs to be updated.  As the D lang
maintainer, can you take care of it?

Thanks!

Simon
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 2/2] D: Sync libiberty with upstream GCC.

Iain Buclaw
On 10 July 2017 at 18:48, Simon Marchi <[hidden email]> wrote:

> On 2017-06-06 22:36, Iain Buclaw wrote:
>>
>> Hi,
>>
>> This patch synchronizes GDB with GCC's copy of libiberty - the D
>> demangling part.
>>
>> Iain.
>>
>> ---
>
>
> Hi Iain,
>
> Starting with this commit, I see this failure in the D tests:
>
> FAIL: gdb.dlang/demangle.exp: _D8demangle4test6__initZ
>
> This command
>
> (gdb) demangle _D8demangle4test6__initZ
> demangle.test.init$
>
> now gives this result
>
> (gdb) demangle _D8demangle4test6__initZ
> initializer for demangle.test
>
> Is it simply that the test needs to be updated.  As the D lang maintainer,
> can you take care of it?
>

Yes, the test just needs to be updated.  I may have to start putting a
sticky note somewhere mentioning to test check-gdb.dlang after
updating libiberty.

Iain.