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]

[PATCH 8/9] [gas] Introduce new section flag: SEC_ELF_OCTETS


All symbols, sizes and relocations in this sections are octets instead of
bytes. Required for DWARF debug sections as DWARF information is
organized in octets, not bytes.

	* dwarf2dbg.c: Define SEC_OCTETS as SEC_ELF_OCTETS if OBJ_ELF.
	(dwarf2_finish): Set section flag SEC_OCTETS for .debug_line,
	.debug_info, .debug_abbrev, .debug_aranges, .debug_str and
	.debug_ranges sections.
	* write.c (maybe_generate_build_notes): Set section flag
	SEC_ELF_OCTETS for .gnu.build.attributes section.
	* frags.c (frag_now_fix): Don't divide by OCTETS_PER_BYTE if
	SEC_ELF_OCTETS is set.
	* symbols.c (resolve_symbol_value): Likewise.

Signed-off-by: Christian Eggers <ceggers@gmx.de>
---
 gas/dwarf2dbg.c | 26 +++++++++++++++++++-------
 gas/frags.c     |  9 ++++++++-
 gas/symbols.c   | 18 ++++++++++++++++--
 gas/write.c     |  6 +++++-
 4 files changed, 48 insertions(+), 11 deletions(-)

diff --git a/gas/dwarf2dbg.c b/gas/dwarf2dbg.c
index dce96033df..6ae65dc874 100644
--- a/gas/dwarf2dbg.c
+++ b/gas/dwarf2dbg.c
@@ -162,6 +162,13 @@
 #define TC_PARSE_CONS_RETURN_NONE BFD_RELOC_NONE
 #endif

+#ifdef OBJ_ELF
+/* On ELF platforms, mark debug sections with SEC_ELF_OCTETS */
+#define SEC_OCTETS SEC_ELF_OCTETS
+#else
+#define SEC_OCTETS 0
+#endif
+
 struct line_entry
 {
   struct line_entry *next;
@@ -2215,7 +2222,7 @@ dwarf2_finish (void)

   /* Create and switch to the line number section.  */
   line_seg = subseg_new (".debug_line", 0);
-  bfd_set_section_flags (line_seg, SEC_READONLY | SEC_DEBUGGING);
+  bfd_set_section_flags (line_seg, SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS);

   /* For each subsection, chain the debug entries together.  */
   for (s = all_segs; s; s = s->next)
@@ -2261,11 +2268,15 @@ dwarf2_finish (void)
       aranges_seg = subseg_new (".debug_aranges", 0);
       str_seg = subseg_new (".debug_str", 0);

-      bfd_set_section_flags (info_seg, SEC_READONLY | SEC_DEBUGGING);
-      bfd_set_section_flags (abbrev_seg, SEC_READONLY | SEC_DEBUGGING);
-      bfd_set_section_flags (aranges_seg, SEC_READONLY | SEC_DEBUGGING);
-      bfd_set_section_flags (str_seg, (SEC_READONLY | SEC_DEBUGGING
-				       | SEC_MERGE | SEC_STRINGS));
+      bfd_set_section_flags (info_seg,
+			      SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS);
+      bfd_set_section_flags (abbrev_seg,
+			      SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS);
+      bfd_set_section_flags (aranges_seg,
+			      SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS);
+      bfd_set_section_flags (str_seg,
+			      SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS
+				       | SEC_MERGE | SEC_STRINGS);
       str_seg->entsize = 1;

       record_alignment (aranges_seg, ffs (2 * sizeof_address) - 1);
@@ -2275,7 +2286,8 @@ dwarf2_finish (void)
       else
 	{
 	  ranges_seg = subseg_new (".debug_ranges", 0);
-	  bfd_set_section_flags (ranges_seg, SEC_READONLY | SEC_DEBUGGING);
+	  bfd_set_section_flags (ranges_seg,
+				 SEC_READONLY | SEC_DEBUGGING | SEC_OCTETS);
 	  record_alignment (ranges_seg, ffs (2 * sizeof_address) - 1);
 	  out_debug_ranges (ranges_seg);
 	}
diff --git a/gas/frags.c b/gas/frags.c
index 2f21b9db20..71e77e5811 100644
--- a/gas/frags.c
+++ b/gas/frags.c
@@ -395,7 +395,14 @@ frag_now_fix_octets (void)
 addressT
 frag_now_fix (void)
 {
-  return frag_now_fix_octets () / OCTETS_PER_BYTE;
+#if OBJ_ELF
+  /* Symbols whose section has SEC_ELF_OCTETS set,
+   * resolve to octets instead of target bytes. */
+  if (now_seg->flags & SEC_ELF_OCTETS)
+    return frag_now_fix_octets ();
+  else
+#endif
+    return frag_now_fix_octets () / OCTETS_PER_BYTE;
 }

 void
diff --git a/gas/symbols.c b/gas/symbols.c
index cff24059e6..b40d528c0d 100644
--- a/gas/symbols.c
+++ b/gas/symbols.c
@@ -1217,7 +1217,14 @@ resolve_symbol_value (symbolS *symp)
       if (local_symbol_resolved_p (locsym))
 	return final_val;

-      final_val += local_symbol_get_frag (locsym)->fr_address / OCTETS_PER_BYTE;
+#if OBJ_ELF
+      /* Symbols whose section has SEC_ELF_OCTETS set,
+       * resolve to octets instead of target bytes. */
+      if (locsym->lsy_section->flags & SEC_ELF_OCTETS)
+	final_val += local_symbol_get_frag (locsym)->fr_address;
+      else
+#endif
+	final_val += local_symbol_get_frag (locsym)->fr_address / OCTETS_PER_BYTE;

       if (finalize_syms)
 	{
@@ -1330,7 +1337,14 @@ resolve_symbol_value (symbolS *symp)
 	  /* Fall through.  */

 	case O_constant:
-	  final_val += symp->sy_frag->fr_address / OCTETS_PER_BYTE;
+#if OBJ_ELF
+	  /* Symbols whose section has SEC_ELF_OCTETS set,
+	   * resolve to octets instead of target bytes. */
+	  if (symp->bsym->section->flags & SEC_ELF_OCTETS)
+	    final_val += symp->sy_frag->fr_address;
+	  else
+#endif
+	    final_val += symp->sy_frag->fr_address / OCTETS_PER_BYTE;
 	  if (final_seg == expr_section)
 	    final_seg = absolute_section;
 	  /* Fall through.  */
diff --git a/gas/write.c b/gas/write.c
index 8f7786eb36..a161520d2c 100644
--- a/gas/write.c
+++ b/gas/write.c
@@ -1960,7 +1960,11 @@ maybe_generate_build_notes (void)
   /* Create a GNU Build Attribute section.  */
   sec = subseg_new (GNU_BUILD_ATTRS_SECTION_NAME, FALSE);
   elf_section_type (sec) = SHT_NOTE;
-  bfd_set_section_flags (sec, SEC_READONLY | SEC_HAS_CONTENTS | SEC_DATA);
+  bfd_set_section_flags (sec, SEC_READONLY | SEC_HAS_CONTENTS | SEC_DATA
+#ifdef OBJ_ELF
+      | SEC_ELF_OCTETS
+#endif
+      );
   bfd_set_section_alignment (sec, 2);

   /* Work out the size of the notes that we will create,
--
2.16.4


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