[PATCH v2] alpha, ld: remove -taso option

Maciej W. Rozycki macro@orcam.me.uk
Mon Jan 20 17:50:32 GMT 2025


On Sat, 18 Jan 2025, Ivan Kokshaysky wrote:

> The -taso switch was quite useful 25 years ago for porting 32-bit
> code with broken integer-pointer casting. Not anymore. The EF_ALPHA_32BIT
> Linux support is going to be dropped in kernel v6.14 [1], NetBSD and OpenBSD
> never had it, so there is no point in keeping the -taso option around.
> 
> Also remove alpha special case that uses -taso from gdb.base/dump.exp
> in gdb testsuite.
> 
> [1] https://lore.kernel.org/all/87jzb2tdb7.fsf_-_@email.froward.int.ebiederm.org
> 
> Signed-off-by: Ivan Kokshaysky <ink@unseen.parts>
> ---
> Changes in v2, suggested by Maciej W. Rozycki:
>   - leave the translations alone;
>   - remove OPTION_TASO from ld/ldlex.h;
>   - add entries to ld/NEWS and binutils/NEWS.

 I forgot we have per-tool-directory NEWS, so thanks for updating ld/NEWS 
and I think we don't want a duplicate entry in binutils/NEWS, which is for 
the other tools.  No need to send v3 I suppose, this can be fixed up at 
patch installation time.

 Your change looks good to me and I've run it through binutils regression 
testing, so as long as Nick is fine with it and someone on the GDB side 
acks the GDB test update I can install it for you once binutils 2.44 have 
been released in ~fortnight time.

  Maciej


More information about the Gdb-patches mailing list