diff options
author | Jan Beulich <jbeulich@suse.com> | 2023-12-15 12:41:21 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2023-12-15 12:41:21 +0100 |
commit | f27110955454028a389d38a7c5017e36b625bdb5 (patch) | |
tree | 4a068b70f43c9a8220a9139aabb1629a81fedf16 /gas | |
parent | d4064aad87f47dac73f9ae2b2c24bae8bb5feabe (diff) | |
download | fsf-binutils-gdb-f27110955454028a389d38a7c5017e36b625bdb5.zip fsf-binutils-gdb-f27110955454028a389d38a7c5017e36b625bdb5.tar.gz fsf-binutils-gdb-f27110955454028a389d38a7c5017e36b625bdb5.tar.bz2 |
ELF: drop "push" parameter from obj_elf_change_section()
No caller outside of obj-elf.c cares about the parameter - drop it by
introducing an obj-elf.c-internal wrapper.
While adding the new function parameter, take the opportunity and change
the adjacent boolean one to "bool".
Diffstat (limited to 'gas')
-rw-r--r-- | gas/config/obj-elf.c | 30 | ||||
-rw-r--r-- | gas/config/obj-elf.h | 2 | ||||
-rw-r--r-- | gas/config/tc-arm.c | 2 | ||||
-rw-r--r-- | gas/config/tc-ia64.c | 2 | ||||
-rw-r--r-- | gas/config/tc-microblaze.c | 12 | ||||
-rw-r--r-- | gas/config/tc-mips.c | 2 | ||||
-rw-r--r-- | gas/config/tc-msp430.c | 2 | ||||
-rw-r--r-- | gas/config/tc-rx.c | 4 | ||||
-rw-r--r-- | gas/config/tc-tic6x.c | 2 |
9 files changed, 34 insertions, 24 deletions
diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c index eaf2d6a..23eff51 100644 --- a/gas/config/obj-elf.c +++ b/gas/config/obj-elf.c @@ -540,14 +540,14 @@ get_section_by_match (bfd *abfd ATTRIBUTE_UNUSED, asection *sec, void *inf) other possibilities, but I don't know what they are. In any case, BFD doesn't really let us set the section type. */ -void -obj_elf_change_section (const char *name, - unsigned int type, - bfd_vma attr, - int entsize, - struct elf_section_match *match_p, - int linkonce, - int push) +static void +change_section (const char *name, + unsigned int type, + bfd_vma attr, + int entsize, + struct elf_section_match *match_p, + bool linkonce, + bool push) { asection *old_sec; segT sec; @@ -820,6 +820,17 @@ obj_elf_change_section (const char *name, #endif } +void +obj_elf_change_section (const char *name, + unsigned int type, + bfd_vma attr, + int entsize, + struct elf_section_match *match_p, + bool linkonce) +{ + change_section (name, type, attr, entsize, match_p, linkonce, false); +} + static bfd_vma obj_elf_parse_section_letters (char *str, size_t len, bool *is_clone, int *inherit, bfd_vma *gnu_attr) @@ -1488,8 +1499,7 @@ obj_elf_section (int push) } } - obj_elf_change_section (name, type, attr, entsize, &match, linkonce, - push); + change_section (name, type, attr, entsize, &match, linkonce, push); if (linked_to_section_index != -1UL) { diff --git a/gas/config/obj-elf.h b/gas/config/obj-elf.h index 683d085..fdd7363 100644 --- a/gas/config/obj-elf.h +++ b/gas/config/obj-elf.h @@ -198,7 +198,7 @@ extern void obj_elf_data (int); extern void obj_elf_text (int); extern void obj_elf_change_section (const char *, unsigned int, bfd_vma, int, struct elf_section_match *, - int, int); + bool); extern void obj_elf_vtable_inherit (int); extern void obj_elf_vtable_entry (int); extern struct fix * obj_elf_get_vtable_inherit (void); diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index 56345ea..1eeb05f 100644 --- a/gas/config/tc-arm.c +++ b/gas/config/tc-arm.c @@ -27775,7 +27775,7 @@ start_unwind_section (const segT text_seg, int idx) } obj_elf_change_section (sec_name, type, flags, 0, &match, - linkonce, 0); + linkonce); /* Set the section link for index tables. */ if (idx) diff --git a/gas/config/tc-ia64.c b/gas/config/tc-ia64.c index 0b97986f..509dc3b 100644 --- a/gas/config/tc-ia64.c +++ b/gas/config/tc-ia64.c @@ -1139,7 +1139,7 @@ obj_elf_vms_common (int ignore ATTRIBUTE_UNUSED) obj_elf_change_section (sec_name, SHT_NOBITS, SHF_ALLOC | SHF_WRITE | SHF_IA_64_VMS_OVERLAID | SHF_IA_64_VMS_GLOBAL, - 0, NULL, 1, 0); + 0, NULL, true); S_SET_VALUE (symbolP, 0); S_SET_SIZE (symbolP, size); diff --git a/gas/config/tc-microblaze.c b/gas/config/tc-microblaze.c index 604cc93..517903c 100644 --- a/gas/config/tc-microblaze.c +++ b/gas/config/tc-microblaze.c @@ -150,7 +150,7 @@ microblaze_s_data (int ignore ATTRIBUTE_UNUSED) { #ifdef OBJ_ELF obj_elf_change_section (".data", SHT_PROGBITS, SHF_ALLOC+SHF_WRITE, - 0, 0, 0, 0); + 0, 0, false); #else s_data (ignore); #endif @@ -163,7 +163,7 @@ microblaze_s_sdata (int ignore ATTRIBUTE_UNUSED) { #ifdef OBJ_ELF obj_elf_change_section (".sdata", SHT_PROGBITS, SHF_ALLOC+SHF_WRITE, - 0, 0, 0, 0); + 0, 0, false); #else s_data (ignore); #endif @@ -282,7 +282,7 @@ microblaze_s_rdata (int localvar) { /* rodata. */ obj_elf_change_section (".rodata", SHT_PROGBITS, SHF_ALLOC, - 0, 0, 0, 0); + 0, 0, false); if (rodata_segment == 0) rodata_segment = subseg_new (".rodata", 0); } @@ -290,7 +290,7 @@ microblaze_s_rdata (int localvar) { /* 1 .sdata2. */ obj_elf_change_section (".sdata2", SHT_PROGBITS, SHF_ALLOC, - 0, 0, 0, 0); + 0, 0, false); } #else s_data (ignore); @@ -303,12 +303,12 @@ microblaze_s_bss (int localvar) #ifdef OBJ_ELF if (localvar == 0) /* bss. */ obj_elf_change_section (".bss", SHT_NOBITS, SHF_ALLOC+SHF_WRITE, - 0, 0, 0, 0); + 0, 0, false); else if (localvar == 1) { /* sbss. */ obj_elf_change_section (".sbss", SHT_NOBITS, SHF_ALLOC+SHF_WRITE, - 0, 0, 0, 0); + 0, 0, false); if (sbss_segment == 0) sbss_segment = subseg_new (".sbss", 0); } diff --git a/gas/config/tc-mips.c b/gas/config/tc-mips.c index 4d40d56..14b969b 100644 --- a/gas/config/tc-mips.c +++ b/gas/config/tc-mips.c @@ -16435,7 +16435,7 @@ s_change_section (int ignore ATTRIBUTE_UNUSED) section_type = SHT_PROGBITS; obj_elf_change_section (section_name, section_type, section_flag, - section_entry_size, 0, 0, 0); + section_entry_size, 0, false); } void diff --git a/gas/config/tc-msp430.c b/gas/config/tc-msp430.c index e3c66a3..8d34fc3 100644 --- a/gas/config/tc-msp430.c +++ b/gas/config/tc-msp430.c @@ -622,7 +622,7 @@ msp430_profiler (int dummy ATTRIBUTE_UNUSED) subseg = now_subseg; /* Now go to .profiler section. */ - obj_elf_change_section (".profiler", SHT_PROGBITS, 0, 0, 0, 0, 0); + obj_elf_change_section (".profiler", SHT_PROGBITS, 0, 0, 0, false); /* Save flags. */ emit_expr (& exp, 2); diff --git a/gas/config/tc-rx.c b/gas/config/tc-rx.c index c011ec1..2a5dca7 100644 --- a/gas/config/tc-rx.c +++ b/gas/config/tc-rx.c @@ -488,7 +488,7 @@ parse_rx_section (char * name) else type = SHT_NOBITS; - obj_elf_change_section (name, type, attr, 0, NULL, false, false); + obj_elf_change_section (name, type, attr, 0, NULL, false); } else /* Try not to redefine a section, especially B_1. */ { @@ -503,7 +503,7 @@ parse_rx_section (char * name) | ((flags & SEC_STRINGS) ? SHF_STRINGS : 0) | ((flags & SEC_THREAD_LOCAL) ? SHF_TLS : 0); - obj_elf_change_section (name, type, attr, 0, NULL, false, false); + obj_elf_change_section (name, type, attr, 0, NULL, false); } bfd_set_section_alignment (now_seg, align); diff --git a/gas/config/tc-tic6x.c b/gas/config/tc-tic6x.c index 6ed61ea..67a662e 100644 --- a/gas/config/tc-tic6x.c +++ b/gas/config/tc-tic6x.c @@ -4662,7 +4662,7 @@ tic6x_start_unwind_section (const segT text_seg, int idx) } obj_elf_change_section (sec_name, type, flags, 0, &match, - linkonce, 0); + linkonce); /* Set the section link for index tables. */ if (idx) |