[PATCH 1/8] gdb.trace: Fix write_inferior_data_ptr on 32-bit big-endian machines.

Marcin Kościelnicki koriakin@0x04.net
Tue Jan 26 13:45:00 GMT 2016


On 26/01/16 14:32, Ulrich Weigand wrote:
> Marcin KoÃ…cielnicki wrote:
>
>> 	* tracepoint.c (write_inferior_data_ptr): Cast to uintptr_t, so that
>> 	it works properly on big-endian machines where sizeof (CORE_ADDR)
>> 	!= sizeof (void *).
>
> This is the same problem that was detected on PowerPC here:
> https://sourceware.org/ml/gdb-patches/2015-03/msg00996.html
>
> Unfortunately that patch, while approved, never got committed.  It still
> seems to be the better solution, since now there's two basically identical
> routines write_inferior_data_ptr and write_inferior_data_pointer for no
> particular reason ...
>
> Bye,
> Ulrich
>


I'll remove the duplicated function.

Is there a chance of getting this patchset commited?  I see I duplicated 
a good chunk of work done there, it's a matter of time before someone 
likewise stumbles upon other already-fixed issues...

Marcin Kościelnicki



More information about the Gdb-patches mailing list