Re: [PATCH v4] gdb: ADI support

Weimin Pan
2017-07-25T22:37:29Z

----- Original Message -----
> From: simon.marchi@polymtl.ca
> To: weimin.pan@oracle.com
> Cc: qiyaoltc@gmail.com, jose.marchesi@oracle.com, gdb-patches@sourceware.org
> Sent: Tuesday, July 25, 2017 1:18:00 PM GMT -08:00 US/Canada Pacific
> Subject: Re: [PATCH v4] gdb: ADI support
>
> ...
> >
> > Given the fact that we need to backport this new feature to older
> > gdb's, e.g.
> > dev-toolset6, one question is do we need to resort to the linked list
> > solution
> > if the std list implementation is not available in older gdb versions?
> > As of now,
> > we have one diversion already - uiout->text ("\n") VS ui_out_text
> > (uiout, "\n").
>
> Are you talking about Red Hat devtoolset? If so, it seems like it
> contains gdb 7.12. As far as I remember, it was possible to build 7.12
> in both C and C++ mode. If you don't need to maintain the ability to
> build as C, you should be able to backport a patch using std::list.

Yes, it is. It will be a big plus if we can apply the same fix for both
old and new versions. Will take a look at std::list or forward_list and
use it over the linked list.

Thanks.