This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [Bug gas/22492] Incorrect note section alignment
- From: Nick Clifton <nickc at redhat dot com>
- To: amodra at gmail dot com <sourceware-bugzilla at sourceware dot org>, bug-binutils at gnu dot org, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Date: Wed, 29 Nov 2017 09:34:55 +0000
- Subject: Re: [Bug gas/22492] Incorrect note section alignment
- Authentication-results: sourceware.org; auth=none
- References: <bug-22492-70@http.sourceware.org/bugzilla/> <bug-22492-70-2lm103JGJN@http.sourceware.org/bugzilla/>
Hi Alan,
> https://sourceware.org/bugzilla/show_bug.cgi?id=22492
> Was the patch in comment #1 posted to the mailing list?
Sorry no - that was my bad.
> There's a small nit..
> I think it should be using record_alignment rather than
> bfd_set_section_alignment.
So it should. I have checked in this patch to fix the problem.
Cheers
Nick
gas/ChangeLog
2017-11-29 Nick Clifton <nickc@redhat.com>
PR 22492
* config/obj-elf.c (obj_elf_version): Use record_alignment rather
than bfd_set_section_alignment.
diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c
index 2d1d1679b8..61c573a5e4 100644
--- a/gas/config/obj-elf.c
+++ b/gas/config/obj-elf.c
@@ -1829,7 +1829,7 @@ obj_elf_version (int ignore ATTRIBUTE_UNUSED)
bfd_set_section_flags (stdoutput,
note_secp,
SEC_HAS_CONTENTS | SEC_READONLY);
- bfd_set_section_alignment (stdoutput, note_secp, 2);
+ record_alignment (note_secp, 2);
/* Process the version string. */
len = strlen (name) + 1;