[PATCH, MIPS] Support shared library debug with MIPS PIE

Richard Sandiford rdsandiford@googlemail.com
Tue Jun 23 18:07:00 GMT 2015


Matthew Fortune <Matthew.Fortune@imgtec.com> writes:
> diff --git a/ld/testsuite/ld-mips-elf/pic-and-nonpic-3b.ad b/ld/testsuite/ld-mips-elf/pic-and-nonpic-3b.ad
> index 227d15d..802bb71 100644
> --- a/ld/testsuite/ld-mips-elf/pic-and-nonpic-3b.ad
> +++ b/ld/testsuite/ld-mips-elf/pic-and-nonpic-3b.ad
> @@ -8,6 +8,7 @@ Dynamic section at offset .* contains .*:
>   0x0000000a \(STRSZ\) .*
>   0x0000000b \(SYMENT\) .*
>   0x70000016 \(MIPS_RLD_MAP\) * 0x80000
> + 0x70000035 \(MIPS_RLD_MAP2\) * 0x3dfc8
>   0x00000015 \(DEBUG\) * 0x0
>   0x00000003 \(PLTGOT\) * 0xa0000
>   0x70000001 \(MIPS_RLD_VERSION\) * 1

For these and similar tests I think it would be better to stub out the
address with .*, so that there's less to update if the gap grows or shrinks
due to some other later change.  It doesn't seem worth matching a particular
value when we have "Dynamic section at offset .*"

Obviously this doesn't apply to your new tests. :-)

Looks good to me with that change.

Thanks,
Richard



More information about the Binutils mailing list