[PATCH 3/4] bfd: pe: Revise pdata sorting conditional

Jan Beulich jbeulich@suse.com
Fri Sep 18 11:16:44 GMT 2026


On 13.09.2026 14:33, Jon Turney wrote:
> --- a/bfd/peXXigen.c
> +++ b/bfd/peXXigen.c
> @@ -3193,7 +3193,7 @@ _bfd_XX_get_symbol_info (bfd * abfd, asymbol *symbol, symbol_info *ret)
>    coff_get_symbol_info (abfd, symbol, ret);
>  }
>  
> -#if !defined(COFF_WITH_pep) && (defined(COFF_WITH_pex64) || defined(COFF_WITH_peAArch64) || defined(COFF_WITH_peLoongArch64) || defined (COFF_WITH_peRiscV64))
> +#if defined(COFF_WITH_pep) || defined(COFF_WITH_pex64) || defined(COFF_WITH_peAArch64)
>  static int
>  sort_pdata (const void *l, const void *r)
>  {

>From the description it's not really becoming clear why Loongarch and RISC-V
would be removed here. Those who have added this may know more than we do,
and trying to do the sorting with a guessed entry size may end up being
better than not sorting at all: In particular when the size there indeed is
12 as well.

Jan


More information about the Binutils mailing list