Re: [PATCH v1 1/8] gdb/linux-tdep: use pid_t consistently for process IDs

[Matthieu Longo <matthieu.longo@arm.com>]

On 05/07/2026 22:22, Tom Tromey wrote:
>>>>>> "Matthieu" == Matthieu Longo <matthieu.longo@arm.com> writes:
> 
> Matthieu> Also change inferior::pid to `pid_t' so that the inferior's process ID
> Matthieu> uses the appropriate type throughout GDB.
> 
> I don't think this is desirable as pid_t is a host type, but the
> inferior's process ID has to possibly represent a target scalar.
> 
> Probably the current use of 'int' here is also wrong, see those recent
> patches I sent and the corresponding bugs.
> 
> Instead it should be ptid_t::pid_type.
> 
> ... and then, ptid_t::pid_type should be LONGEST or, even better, a new
> enum with LONGEST as the base type, to avoid doing any arithmetic on it.
> 
> I started work on this a while ago but it's a long slog & I haven't
> finished.
> 
> Tom

I will remove that from the patch series.
Since you already started working on it, it is probably better to tackle it as a whole.
Thanks for your review.

Matthieu