[PATCH] LD/PE: Remove remains of MIPS target support
Maciej W. Rozycki
macro@orcam.me.uk
Thu Aug 21 13:14:30 GMT 2025
On Mon, 18 Aug 2025, Jan Beulich wrote:
> >> --- binutils-gdb.orig/ld/pe-dll.c
> >> +++ binutils-gdb/ld/pe-dll.c
> >> @@ -247,10 +247,9 @@ static const autofilter_entry_type autof
> >>
> >> #define PE_ARCH_i386 1
> >> #define PE_ARCH_sh 2
> >> -#define PE_ARCH_mips 3
> >> -#define PE_ARCH_arm 4
> >> -#define PE_ARCH_arm_wince 5
> >> -#define PE_ARCH_aarch64 6
> >> +#define PE_ARCH_arm 3
> >> +#define PE_ARCH_arm_wince 4
> >> +#define PE_ARCH_aarch64 5
> >
> > If this is purely internal (and it certainly looks like it), perhaps
> > replacing the #defines with an enum would make more sense here, so
> > that next time we don't get more churn here.
>
> Alternatively, to avoid the churn here, value 3 could remain unused for
> now, and my patch could then re-purpose it for PE_ARCH_mcore.
Seems like a hack to me, but I like the idea of using an enum instead.
Maciej
More information about the Binutils
mailing list