From: Luis Machado Date: Wed, 2 Jul 2014 10:59:02 +0000 (+0100) Subject: This testcase currently does not handle powerpc branches. It kinda X-Git-Tag: upstream/2.25~928 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2b239efbec446a0c213710b7c5258f2926ef78a4;p=platform%2Fupstream%2Flinaro-binutils.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 * gdb.trace/entry-values.exp: Handle powerpc-specific branch instruction. --- diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 90be171..af1dc66 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 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" }