[patch]: Support coff noread flag for segments
Kai Tietz
ktietz70@googlemail.com
Sun May 17 09:13:00 GMT 2009
2009/5/17 Dave Korn <dave.korn.cygwin@googlemail.com>:
> Kai Tietz wrote:
>
>>
>> Sorry, I copied it from console output and therefore things got
>> corrupt. Patch is appended to this mail as attachment.
>
> The patch itself is fine now, but I spotted something missing. You need to
> update the documentation of the ".section" pseudo-op in as.texinfo to explain
> the new letter. Ok once you've added documentation.
>
> Also, please give serious consideration to adding even just a single trivial
> testcase that would verify the new functionality continues to work and check
> with "objdump -h" that it sets the correct flags in the header. I can help
> with that if you're not familiar with the testsuite infrastructure.
>
> cheers,
> DaveK
>
>
So here is the patch with documentation changes, too. Possibly your
testcase for .drectve we could use here in gas, too?
Cheers,
Kai
--
| (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination
-------------- next part --------------
A non-text attachment was scrubbed...
Name: noread.diff
Type: text/x-diff
Size: 1699 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20090517/d95c5db8/attachment.bin>
More information about the Binutils
mailing list