From 007801eeb19ed565eb4dc4020831f6d293f46930 Mon Sep 17 00:00:00 2001 From: James Clarke Date: Wed, 10 Aug 2016 09:47:45 -0700 Subject: [PATCH] Fix improper handling of R_SPARC_GOTDATA_OP_LOX10 relocation. The fall-through in Target_sparc::Relocate::relocate for R_SPARC_GOTDATA_OP_LOX10 is currently R_SPARC_GOT13, but should clearly be R_SPARC_GOT10. GCC has been seen to emit a sethi/xor rather than a sethi/or sequence to load a 32-bit immediate, but if R_SPARC_GOT13 is used then bits 10-12 get zeroed out as both the sethi and xor immediates contain them. gold/ PR gold/20442 * sparc.cc (Target_sparc::Relocate::relocate): R_SPARC_GOTDATA_OP_LOX10 should fall back on R_SPARC_GOT10, not R_SPARC_GOT13. --- gold/ChangeLog | 6 ++++++ gold/sparc.cc | 14 +++++++------- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index cca5455..d0a670b 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,5 +1,11 @@ 2016-08-10 James Clarke + PR gold/20442 + * sparc.cc (Target_sparc::Relocate::relocate): R_SPARC_GOTDATA_OP_LOX10 + should fall back on R_SPARC_GOT10, not R_SPARC_GOT13. + +2016-08-10 James Clarke + PR gold/20441 * sparc.cc (Target_sparc::Scan::check_non_pic): Allow R_SPARC_32 on sparc64. diff --git a/gold/sparc.cc b/gold/sparc.cc index d9a8ef4..8e66b77 100644 --- a/gold/sparc.cc +++ b/gold/sparc.cc @@ -3469,6 +3469,13 @@ Target_sparc::Relocate::relocate( Reloc::lo10(view, object, psymval, addend); break; + case elfcpp::R_SPARC_GOTDATA_OP_LOX10: + if (gdop_valid) + { + Reloc::gdop_lox10(view, got_offset); + break; + } + /* Fall through. */ case elfcpp::R_SPARC_GOT10: Reloc::lo10(view, got_offset, addend); break; @@ -3487,13 +3494,6 @@ Target_sparc::Relocate::relocate( } break; - case elfcpp::R_SPARC_GOTDATA_OP_LOX10: - if (gdop_valid) - { - Reloc::gdop_lox10(view, got_offset); - break; - } - /* Fall through. */ case elfcpp::R_SPARC_GOT13: Reloc::rela32_13(view, got_offset, addend); break; -- 2.7.4