diff options
| author | Cary Coutant <ccoutant@google.com> | 2014-07-08 22:34:27 -0700 |
|---|---|---|
| committer | Cary Coutant <ccoutant@google.com> | 2014-07-08 22:52:37 -0700 |
| commit | e051745c838bf29e564fb2665339f97c8383b9e8 (patch) | |
| tree | f95cb0a6f37660cfa33bacef128b335f4dc678e3 /gold/symtab.h | |
| parent | ecf3e831f71257e8ff7bf794f394f23b26d75137 (diff) | |
| download | binutils-gdb-e051745c838bf29e564fb2665339f97c8383b9e8.tar.gz | |
Fix --defsym to copy symbol attributes.
Alan Modra committed a patch to Gnu ld to fix a problem encountered on
PPC where the --defsym option wasn't copying the st_other bits to the
newly-defined symbol.
https://sourceware.org/ml/binutils/2014-07/msg00094.html
Gold has the same problem, and additionally wasn't copying the symbol type.
This patch fixes both problems, by copying the symbol type, visibility, and
the remaining st_other bits to the new symbol for --defsym=sym1=sym2
assignments.
gold/
* expression.cc (struct Expression::Expression_eval_info): Add
new fields type_pointer, vis_pointer, and nonvis_pointer.
(Expression::eval_maybe_dot): Add type_pointer, vis_pointer, and
nonvis_pointer parameters. Adjust all calls.
(Symbol_expression::value): Update type, visibility, and nonvis bits
in caller.
* script.cc (Symbol_assignment::sized_finalize): Update type,
visibility, and remaining st_other bits for new symbol.
* script.h: (Expression::eval_maybe_dot): Add type_pointer,
vis_pointer, and nonvis_pointer parameters.
* symtab.h (Symbol::set_type): New method.
* testsuite/Makefile.am (defsym_test): New test.
* testsuite/Makefile.in: Regenerate.
* testsuite/defsym_test.c: New file.
* testsuite/defsym_test.sh: New file.
Diffstat (limited to 'gold/symtab.h')
| -rw-r--r-- | gold/symtab.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/gold/symtab.h b/gold/symtab.h index ab5b5f97d7f..7984dd6ff84 100644 --- a/gold/symtab.h +++ b/gold/symtab.h @@ -215,6 +215,11 @@ class Symbol type() const { return this->type_; } + // Set the symbol type. + void + set_type(elfcpp::STT type) + { this->type_ = type; } + // Return true for function symbol. bool is_func() const |