This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[PATCH] MIPS gas: Check for ELF format in md_section_align at runtime
- From: Thiemo Seufer <ths at networkno dot de>
- To: binutils at sourceware dot org
- Date: Tue, 1 Aug 2006 09:02:45 +0100
- Subject: [PATCH] MIPS gas: Check for ELF format in md_section_align at runtime
Hello All,
I applied the appended patch. Gas can do a runtime check for ELF in
this place, no need for a define.
Thiemo
2006-08-01 Thiemo Seufer <ths@mips.com>
* config/tc-mips.c (md_section_align): Check ELF-ness at runtime.
Index: gas/config/tc-mips.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-mips.c,v
retrieving revision 1.353
diff -u -p -r1.353 tc-mips.c
--- gas/config/tc-mips.c 1 Aug 2006 05:49:02 -0000 1.353
+++ gas/config/tc-mips.c 1 Aug 2006 07:51:39 -0000
@@ -13042,16 +13042,17 @@ md_section_align (asection *seg, valueT
{
int align = bfd_get_section_alignment (stdoutput, seg);
-#ifdef OBJ_ELF
- /* We don't need to align ELF sections to the full alignment.
- However, Irix 5 may prefer that we align them at least to a 16
- byte boundary. We don't bother to align the sections if we are
- targeted for an embedded system. */
- if (strcmp (TARGET_OS, "elf") == 0)
- return addr;
- if (align > 4)
- align = 4;
-#endif
+ if (IS_ELF)
+ {
+ /* We don't need to align ELF sections to the full alignment.
+ However, Irix 5 may prefer that we align them at least to a 16
+ byte boundary. We don't bother to align the sections if we
+ are targeted for an embedded system. */
+ if (strcmp (TARGET_OS, "elf") == 0)
+ return addr;
+ if (align > 4)
+ align = 4;
+ }
return ((addr + (1 << align) - 1) & (-1 << align));
}