From 2b239efbec446a0c213710b7c5258f2926ef78a4 Mon Sep 17 00:00:00 2001 From: Luis Machado Date: Wed, 2 Jul 2014 11:59:02 +0100 Subject: [PATCH] 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 * gdb.trace/entry-values.exp: Handle powerpc-specific branch instruction. --- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.trace/entry-values.exp | 2 ++ 2 files changed, 7 insertions(+) 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 + + * gdb.trace/entry-values.exp: Handle powerpc-specific branch + instruction. + 2014-06-30 Mark Wielaard * 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" } -- 2.30.2