[RFA] Define DT_MIPS_RLD_MAP if not already defined.
Pedro Alves
palves@redhat.com
Wed May 30 18:50:00 GMT 2012
On 05/30/2012 07:35 PM, Thiago Jung Bauermann wrote:
> On Wed, 2012-05-30 at 19:03 +0100, Pedro Alves wrote:
>> On 05/30/2012 05:40 AM, Thiago Jung Bauermann wrote:
>>> Hello,
>>>
>>> Android doesn't support MIPS, so understandably there's no
>>> DT_MIPS_RLD_MAP in its header files. This patch allows gdbserver to be
>>> compiled for Android.
>>
>> Then why not '#ifdef DT_MIPS_RLD_MAP' out the bits that use DT_MIPS_RLD_MAP ?
>> It's just dead code on non-MIPS ports.
>
> I thought #ifdef'ing platform-specific code was discouraged in GDB. I
> can certainly do that though.
It is, mostly in common code. But this is native code. The alternative
is to abstract it in the linux_target_ops interface, but that's quite
overkill for this. Plenty of other #ifdefs in linux-low.c.
--
Pedro Alves
More information about the Gdb-patches
mailing list