This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Cast to uintptr_t when calling ptrace32 on aix
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: GDB Patches <gdb-patches at sourceware dot org>, Joel Brobecker <brobecker at adacore dot com>, Ulrich Weigand <uweigand at de dot ibm dot com>
- Date: Sun, 5 Jan 2014 20:21:01 -0500
- Subject: Re: [PATCH] Cast to uintptr_t when calling ptrace32 on aix
- Authentication-results: sourceware.org; auth=none
- References: <CAGWvnykbAX1aY+qeXT0GR6rBevK6ao2Us9nt4UmwUvtdzkbK1A at mail dot gmail dot com> <52CA02FE dot 6060804 at codesourcery dot com>
I meant it would be nice to encapsulate this in one file instead of
defining ptrace32, ptrace64, ptracex, etc. multiple times in multiple
files. It appears that you are performing the same cast as Joel, so it
should be correct. It would be nice to use a consistent syntax. But it
would be even nicer to do this only once.
Thanks David
On Sun, Jan 5, 2014 at 8:12 PM, Yao Qi <yao@codesourcery.com> wrote:
> On 01/06/2014 12:22 AM, David Edelsohn wrote:
>> Thanks for uncovering this and creating a patch. Any patch like this
>> should be regression tested.
>>
>
> OK, I'll test it on gcc111.
>
>> I thought that these issues had been addressed in Joel's patches from
>> last August.
>>
>> https://sourceware.org/ml/gdb-patches/2013-08/msg00657.html
>>
>> Why do we need to go through this delicate dance in slightly different
>> ways in multiple files?
>
> IMO, we do it in the same way, convert address to uintptr_t first and
> then to "addr_ptr" (long long). Joel's is the implicit conversion and
> mine is an explicit one.
>
> --
> Yao (éå)