This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH] Handle powerpc branch instruction on gdb.trace/entry-values.exp
- From: Luis Machado <lgustavo at codesourcery dot com>
- To: "'gdb-patches at sourceware dot org'" <gdb-patches at sourceware dot org>
- Date: Tue, 1 Jul 2014 15:21:49 +0200
- Subject: [PATCH] Handle powerpc branch instruction on gdb.trace/entry-values.exp
- Authentication-results: sourceware.org; auth=none
- Reply-to: <lgustavo at codesourcery dot com>
Hi,
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.
I see full passes and 1 unsupported test now (due to lack of tracepoint
support).
Luis
2014-07-01 Luis Machado <lgustavo@codesourcery.com>
* gdb.trace/entry-values.exp: Handle powerpc-specific branch
instruction.
diff --git a/gdb/testsuite/gdb.trace/entry-values.exp b/gdb/testsuite/gdb.trace/entry-values.exp
index f10ffa6..d648bd1 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"
}