From: Luis Machado <lgustavo@codesourcery.com>
Date: Wed, 2 Jul 2014 10:59:02 +0000 (+0100)
Subject: This testcase currently does not handle powerpc branches. It kinda
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=2b239efbec446a0c213710b7c5258f2926ef78a4;p=binutils-gdb.git

This testcase currently does not handle powerpc branches. It kinda
does in a way, because the arm/aarch64 branch instruction is the
same as powerpc's, but the target triplet pattern is not there.

In summary, the testcase fails to locate the branch offset and causes
a failure and the early termination of the test.

The following patch adds a separate conditional block for powerpc (to keep
things organized), allowing the testcase to continue.

2014-07-02  Luis Machado  <lgustavo@codesourcery.com>

	* gdb.trace/entry-values.exp: Handle powerpc-specific branch
	instruction.
---

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 90be171001e..af1dc6632f9 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2014-07-02  Luis Machado  <lgustavo@codesourcery.com>
+
+	* gdb.trace/entry-values.exp: Handle powerpc-specific branch
+	instruction.
+
 2014-06-30  Mark Wielaard  <mjw@redhat.com>
 
 	* gdb.base/constvars.c (violent, violet, vips, virgen, vulgar,
diff --git a/gdb/testsuite/gdb.trace/entry-values.exp b/gdb/testsuite/gdb.trace/entry-values.exp
index f10ffa6b6e0..d648bd1fcc8 100644
--- a/gdb/testsuite/gdb.trace/entry-values.exp
+++ b/gdb/testsuite/gdb.trace/entry-values.exp
@@ -79,6 +79,8 @@ if { [istarget "arm*-*-*"] || [istarget "aarch64*-*-*"] } {
     set call_insn "bl"
 } elseif { [istarget "s390*-*-*"] } {
     set call_insn "brasl"
+} elseif { [istarget "powerpc*-*-*"] } {
+    set call_insn "bl"
 } else {
     set call_insn "call"
 }