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 PE-COFF]: Some minor nits


Kai Tietz wrote:
> Hello,

  Hi Kai,

> Some minor nits I found in coffcode.h. One is that debugging sections have 
> to have readonly flag set (abstracted by the section flags of '.debug' in 
> pe-coff spec).

  Can you point us at a reference for this?  And have you run the GDB
testsuite to see if it causes any changes?  (As I discovered recently,
sometimes the other tools have come to depend on these minor non-compliances
of ours!)

> Secondly a clean-up about NOREAD sections. 

  http://www.national.com/rap/files/datasheet.pdf :)

> I introduced a helper flag for 
> section's flags to indicated, that a section has no memory flag set,
> I tested this patch for i686-pc-cygwin and for x86_64-pc-ming32 and saw no 
> regressions on testrun.

  Does anything actually currently use NOREAD?

    cheers,
      DaveK


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