diff options
| author | Jiong Wang <jiong.wang@arm.com> | 2017-07-03 17:15:26 +0100 |
|---|---|---|
| committer | Jiong Wang <jiong.wang@arm.com> | 2017-07-04 09:24:10 +0100 |
| commit | 564e46b795ea9ef95ec662f01139a97717de1603 (patch) | |
| tree | b84ed01719012cf740270ee0380daf1fdea30419 /bfd/ChangeLog | |
| parent | 0041f7df741692954ebc112c69e5326afb0115ba (diff) | |
[AArch64] Remove useless and incorrect assertion
The outer caller elf_link_output_extsym in elflink.c is a traverse function on
all external symbol, and it will only call *finish_dynamic_symbol if some
conditions is meet. It is executed conditionally.
If the condition to trigger that assertion is satisified, it then won't satify
the outer check in finish_dynamic_symbol, so *finish_dynamic_symbol won't be
called that the assertion is expected to be dead code.
If elf_link_output_extsym is a traverse function that unconditionally called
on external symbols decided to be exported, then an assertion to make sure these
symbols are in sane status might make sense.
bfd/
* elfnn-aarch64.c (elfNN_aarch64_finish_dynamic_symbol): Remove the
sanity check at the head of this function.
Diffstat (limited to 'bfd/ChangeLog')
| -rw-r--r-- | bfd/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index b2b028fe59..810a555484 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2017-07-04 Jiong Wang <jiong.wang@arm.com> + + * elfnn-aarch64.c (elfNN_aarch64_finish_dynamic_symbol): Remove the + sanity check at the head of this function. + 2017-07-04 Jan Kratochvil <jan.kratochvil@redhat.com> * dwarf2.c (struct dwarf2_debug): Add fields dwarf_line_str_buffer and |