Re: [patch] Smaller gdb: -rdynamic -> -Wl,--dynamic-list

Doug Evans
2010-06-08T16:12:54Z

On Tue, Jun 8, 2010 at 9:07 AM, Jan Kratochvil
<jan.kratochvil@redhat.com> wrote:
> On Tue, 08 Jun 2010 17:22:54 +0200, Doug Evans wrote:
>> On Sat, Jun 5, 2010 at 5:45 AM, Jan Kratochvil <jan.kratochvil@redhat.com> wrote:
>> > gdb/
>> >        * configure.ac <"${have_libpython}" != no>: New workaround of
>> >        python#4434.
>> >        * configure: Regenerate.
>> > gdb/testsuite/
>> >        * gdb.python/python.exp (pythonX.Y/lib-dynload/*.so): New.
>>
>> I tested this and it fixes things.  Thanks.
>
> Considering it as your check-in approval, going to check it in in a moment.

Yep.

>> There is the issue of using AC_RUN_IFELSE and cross-compilation.
>> Python doesn't have good support for cross-compilation anyways, so
>> it's fine with me to punt on this for now.
>
> In this case the current code fallbacks to the larger but more compatible
> -rdynamic (and thus this my recent optimization does not take place).
>
> +         [dynamic_list=true], [], [true])
> ->                                 ^^^^^^
> +       if test "$cross_compiling" = yes; then :
> +  true
> ->
> kept there the pre-set value:
>        dynamic_list=false

Yeah, saw that. "works for me".