diff options
| author | James Clarke <jrtc27@jrtc27.com> | 2017-05-13 08:01:15 -0700 |
|---|---|---|
| committer | Cary Coutant <ccoutant@gmail.com> | 2017-05-13 08:01:15 -0700 |
| commit | 67f46fed806465c005b15c97d33947f4c0abb25c (patch) | |
| tree | b9b0d5e927e3a8c9c150167075574d1631b16aff | |
| parent | adf3dde510088ef8dc46d04df05baf36adb0ed1e (diff) | |
| download | gdb-67f46fed806465c005b15c97d33947f4c0abb25c.zip gdb-67f46fed806465c005b15c97d33947f4c0abb25c.tar.gz gdb-67f46fed806465c005b15c97d33947f4c0abb25c.tar.bz2 | |
Fix assertion failure relaxing TLS for position-independent executables.
gold/
PR gold/21444
* gold.cc (Target_sparc::Relocate::relocate_tls): Local
variables are final for position-independent executables. This
has to be consistent with Target_sparc::Scan::local otherwise
they will disagree as to whether local-exec is used.
| -rw-r--r-- | gold/ChangeLog | 8 | ||||
| -rw-r--r-- | gold/sparc.cc | 2 |
2 files changed, 9 insertions, 1 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index 51d6a0f..0f1f4e5 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,11 @@ +2017-05-13 James Clarke <jrtc27@jrtc27.com> + + PR gold/21444 + * gold.cc (Target_sparc::Relocate::relocate_tls): Local + variables are final for position-independent executables. This + has to be consistent with Target_sparc::Scan::local otherwise + they will disagree as to whether local-exec is used. + 2017-05-12 Igor Kudrin <ikudrin@accesssoftek.com> PR gold/21430 diff --git a/gold/sparc.cc b/gold/sparc.cc index a9cb93a..54c7c33 100644 --- a/gold/sparc.cc +++ b/gold/sparc.cc @@ -3730,7 +3730,7 @@ Target_sparc<size, big_endian>::Relocate::relocate_tls( const bool is_final = (gsym == NULL - ? !parameters->options().output_is_position_independent() + ? !parameters->options().shared() : gsym->final_value_is_known()); const tls::Tls_optimization optimized_type = optimize_tls_reloc(is_final, r_type); |