aboutsummaryrefslogtreecommitdiff
path: root/gold
diff options
context:
space:
mode:
authorMarcin Kościelnicki <koriakin@0x04.net>2016-05-06 23:02:43 +0200
committerMarcin Kościelnicki <koriakin@0x04.net>2016-05-30 19:06:45 +0200
commit40d85a7f9c3446db070bd792d9f2106dc9adaf8c (patch)
treec39b2ee458e31b82c8a8ec919d06f02431cf2eeb /gold
parent825c8ef28fc669bdf4eab64b43d7a64761fbd677 (diff)
downloadgdb-40d85a7f9c3446db070bd792d9f2106dc9adaf8c.zip
gdb-40d85a7f9c3446db070bd792d9f2106dc9adaf8c.tar.gz
gdb-40d85a7f9c3446db070bd792d9f2106dc9adaf8c.tar.bz2
gold/s390: Fix compilation on gcc 4.4
gold/ChangeLog: PR/19960 * s390.cc (Target_s390::ss_code_st_r14): Removed. (Target_s390::ss_code_l_r14): Removed. (Target_s390::ss_code_ear): Removed. (Target_s390::ss_code_c): Removed. (Target_s390::ss_match_st_r14): New function. (Target_s390::ss_match_l_r14): New function. (Target_s390::ss_match_mcount): Call ss_match_{l,st}_r14 instead of matching code directly. (Target_s390::ss_match_ear): New function. (Target_s390::ss_match_c): New function. (Target_s390::do_calls_non_split): Call ss_match_{ear,c} instead of matching code directly.
Diffstat (limited to 'gold')
-rw-r--r--gold/ChangeLog16
-rw-r--r--gold/s390.cc212
2 files changed, 162 insertions, 66 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog
index b2a374f..90b10e5 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,19 @@
+2016-05-30 Marcin Kościelnicki <koriakin@0x04.net>
+
+ PR/19960
+ * s390.cc (Target_s390::ss_code_st_r14): Removed.
+ (Target_s390::ss_code_l_r14): Removed.
+ (Target_s390::ss_code_ear): Removed.
+ (Target_s390::ss_code_c): Removed.
+ (Target_s390::ss_match_st_r14): New function.
+ (Target_s390::ss_match_l_r14): New function.
+ (Target_s390::ss_match_mcount): Call ss_match_{l,st}_r14 instead
+ of matching code directly.
+ (Target_s390::ss_match_ear): New function.
+ (Target_s390::ss_match_c): New function.
+ (Target_s390::do_calls_non_split): Call ss_match_{ear,c} instead
+ of matching code directly.
+
2016-05-19 Cary Coutant <ccoutant@gmail.com>
PR gold/19823
diff --git a/gold/s390.cc b/gold/s390.cc
index 5245eca..c496018 100644
--- a/gold/s390.cc
+++ b/gold/s390.cc
@@ -757,22 +757,30 @@ class Target_s390 : public Sized_target<size, true>
Layout *layout_;
// Code sequences for -fsplit-stack matching.
- static const unsigned char ss_code_st_r14[];
- static const unsigned char ss_code_l_r14[];
static const unsigned char ss_code_bras_8[];
static const unsigned char ss_code_l_basr[];
static const unsigned char ss_code_a_basr[];
- static const unsigned char ss_code_ear[];
- static const unsigned char ss_code_c[];
static const unsigned char ss_code_larl[];
static const unsigned char ss_code_brasl[];
static const unsigned char ss_code_jg[];
static const unsigned char ss_code_jgl[];
// Variable code sequence matchers for -fsplit-stack.
+ bool ss_match_st_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const;
+ bool ss_match_l_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const;
bool ss_match_mcount(unsigned char* view,
section_size_type view_size,
section_offset_type *offset) const;
+ bool ss_match_ear(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const;
+ bool ss_match_c(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const;
bool ss_match_l(unsigned char* view,
section_size_type view_size,
section_offset_type *offset,
@@ -4321,30 +4329,6 @@ Target_s390<size>::do_is_call_to_non_split(const Symbol* sym,
// Code sequences to match below.
-template<>
-const unsigned char
-Target_s390<32>::ss_code_st_r14[] = {
- 0x50, 0xe0, 0xf0, 0x04, // st %r14, 4(%r15)
-};
-
-template<>
-const unsigned char
-Target_s390<64>::ss_code_st_r14[] = {
- 0xe3, 0xe0, 0xf0, 0x08, 0x00, 0x24 // stg %r14, 8(%r15)
-};
-
-template<>
-const unsigned char
-Target_s390<32>::ss_code_l_r14[] = {
- 0x58, 0xe0, 0xf0, 0x04, // l %r14, 4(%r15)
-};
-
-template<>
-const unsigned char
-Target_s390<64>::ss_code_l_r14[] = {
- 0xe3, 0xe0, 0xf0, 0x08, 0x00, 0x04 // lg %r14, 8(%r15)
-};
-
template<int size>
const unsigned char
Target_s390<size>::ss_code_bras_8[] = {
@@ -4368,32 +4352,6 @@ Target_s390<size>::ss_code_a_basr[] = {
0x0d, 0xee, // basr %r14, %r14
};
-template<>
-const unsigned char
-Target_s390<32>::ss_code_ear[] = {
- 0xb2, 0x4f, 0x00, 0x10, // ear %r1, %a0
-};
-
-template<>
-const unsigned char
-Target_s390<64>::ss_code_ear[] = {
- 0xb2, 0x4f, 0x00, 0x10, // ear %r1, %a0
- 0xeb, 0x11, 0x00, 0x20, 0x00, 0x0d, // sllg %r1,%r1,32
- 0xb2, 0x4f, 0x00, 0x11, // ear %r1, %a1
-};
-
-template<>
-const unsigned char
-Target_s390<32>::ss_code_c[] = {
- 0x59, 0xf0, 0x10, 0x20, // c %r15, 0x20(%r1)
-};
-
-template<>
-const unsigned char
-Target_s390<64>::ss_code_c[] = {
- 0xe3, 0xf0, 0x10, 0x38, 0x00, 0x20, // cg %r15, 0x38(%r1)
-};
-
template<int size>
const unsigned char
Target_s390<size>::ss_code_larl[] = {
@@ -4418,6 +4376,70 @@ Target_s390<size>::ss_code_jgl[] = {
0xc0, 0x44, // jgl ...
};
+template<>
+bool
+Target_s390<32>::ss_match_st_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_st_r14[] = {
+ 0x50, 0xe0, 0xf0, 0x04, // st %r14, 4(%r15)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_st_r14,
+ sizeof ss_code_st_r14))
+ return false;
+ *offset += sizeof ss_code_st_r14;
+ return true;
+}
+
+template<>
+bool
+Target_s390<64>::ss_match_st_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_st_r14[] = {
+ 0xe3, 0xe0, 0xf0, 0x08, 0x00, 0x24 // stg %r14, 8(%r15)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_st_r14,
+ sizeof ss_code_st_r14))
+ return false;
+ *offset += sizeof ss_code_st_r14;
+ return true;
+}
+
+template<>
+bool
+Target_s390<32>::ss_match_l_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_l_r14[] = {
+ 0x58, 0xe0, 0xf0, 0x04, // l %r14, 4(%r15)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_l_r14,
+ sizeof ss_code_l_r14))
+ return false;
+ *offset += sizeof ss_code_l_r14;
+ return true;
+}
+
+template<>
+bool
+Target_s390<64>::ss_match_l_r14(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_l_r14[] = {
+ 0xe3, 0xe0, 0xf0, 0x08, 0x00, 0x04 // lg %r14, 8(%r15)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_l_r14,
+ sizeof ss_code_l_r14))
+ return false;
+ *offset += sizeof ss_code_l_r14;
+ return true;
+}
+
template<int size>
bool
Target_s390<size>::ss_match_mcount(unsigned char* view,
@@ -4428,10 +4450,8 @@ Target_s390<size>::ss_match_mcount(unsigned char* view,
section_offset_type myoff = *offset;
// First, look for the store instruction saving %r14.
- if (!this->match_view_u(view, view_size, myoff, ss_code_st_r14,
- sizeof ss_code_st_r14))
+ if (!this->ss_match_st_r14(view, view_size, &myoff))
return false;
- myoff += sizeof ss_code_st_r14;
// Now, param load and the actual call.
if (this->match_view_u(view, view_size, myoff, ss_code_larl,
@@ -4468,10 +4488,8 @@ Target_s390<size>::ss_match_mcount(unsigned char* view,
return false;
// Finally, a load bringing %r14 back.
- if (!this->match_view_u(view, view_size, myoff, ss_code_l_r14,
- sizeof ss_code_l_r14))
+ if (!this->ss_match_l_r14(view, view_size, &myoff))
return false;
- myoff += sizeof ss_code_l_r14;
// Found it.
*offset = myoff;
@@ -4480,6 +4498,72 @@ Target_s390<size>::ss_match_mcount(unsigned char* view,
template<>
bool
+Target_s390<32>::ss_match_ear(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_ear[] = {
+ 0xb2, 0x4f, 0x00, 0x10, // ear %r1, %a0
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_ear,
+ sizeof ss_code_ear))
+ return false;
+ *offset += sizeof ss_code_ear;
+ return true;
+}
+
+template<>
+bool
+Target_s390<64>::ss_match_ear(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_ear[] = {
+ 0xb2, 0x4f, 0x00, 0x10, // ear %r1, %a0
+ 0xeb, 0x11, 0x00, 0x20, 0x00, 0x0d, // sllg %r1,%r1,32
+ 0xb2, 0x4f, 0x00, 0x11, // ear %r1, %a1
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_ear,
+ sizeof ss_code_ear))
+ return false;
+ *offset += sizeof ss_code_ear;
+ return true;
+}
+
+template<>
+bool
+Target_s390<32>::ss_match_c(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_c[] = {
+ 0x59, 0xf0, 0x10, 0x20, // c %r15, 0x20(%r1)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_c,
+ sizeof ss_code_c))
+ return false;
+ *offset += sizeof ss_code_c;
+ return true;
+}
+
+template<>
+bool
+Target_s390<64>::ss_match_c(unsigned char* view,
+ section_size_type view_size,
+ section_offset_type *offset) const
+{
+ static const unsigned char ss_code_c[] = {
+ 0xe3, 0xf0, 0x10, 0x38, 0x00, 0x20, // cg %r15, 0x38(%r1)
+ };
+ if (!this->match_view_u(view, view_size, *offset, ss_code_c,
+ sizeof ss_code_c))
+ return false;
+ *offset += sizeof ss_code_c;
+ return true;
+}
+
+template<>
+bool
Target_s390<32>::ss_match_l(unsigned char* view,
section_size_type view_size,
section_offset_type *offset,
@@ -4652,18 +4736,14 @@ Target_s390<size>::do_calls_non_split(Relobj* object, unsigned int shndx,
// First, figure out if there's a conditional call by looking for the
// extract-tp, add, cmp sequence.
- if (this->match_view_u(view, view_size, curoffset, ss_code_ear,
- sizeof ss_code_ear))
+ if (this->ss_match_ear(view, view_size, &curoffset))
{
// Found extract-tp, now look for an add and compare.
- curoffset += sizeof ss_code_ear;
conditional = true;
- if (this->match_view_u(view, view_size, curoffset, ss_code_c,
- sizeof ss_code_c))
+ if (this->ss_match_c(view, view_size, &curoffset))
{
// Found a direct compare of stack pointer with the guard,
// we're done here.
- curoffset += sizeof ss_code_c;
}
else if (this->ss_match_l(view, view_size, &curoffset, &guard_reg))
{