This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch]: dump of PE+ x64 pdata section

H.J. Lu wrote:

>>>
>>  Shouldn't _bfd_pep_print_x64_pdata be renamed to
>>
>> bfd_boolean _bfd_pex64_print_pdata (bfd *, void *);
>>
>>
> 
> Or this function should be moved to coff-x86-64.c.

  I think the naming is reasonble.  PE+ is an object file format; x86_64 may
be currently the only CPU that uses it, but it's conceivable there might be
non-x64 pdata sections in some future non-x64 architecture that uses the same
format.

  Moving it might well be a good idea though.  The ARM one,
_bfd_XX_print_ce_compressed_pdata probably doesn't really belong there either.

    cheers,
      DaveK
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]