From ff34e6ae3867a5686b53ce3b69584c1c3b553d7d Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Mon, 1 Oct 2018 12:02:57 -0400 Subject: [PATCH] Fix is_amd64_regs_target check in i386-avx512.exp As reported by Jan here: https://sourceware.org/ml/gdb-patches/2018-09/msg00831.html the check that sets the number of available registers is backwards. gdb/testsuite/ChangeLog: * gdb.arch/i386-avx512.exp: Fix setting of nr_regs based on is_amd64_regs_target. --- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.arch/i386-avx512.exp | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 3e36b3a8bb1..502c5bf0bbb 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2018-10-01 Simon Marchi + + * gdb.arch/i386-avx512.exp: Fix setting of nr_regs based on + is_amd64_regs_target. + 2018-10-01 Alan Hayward * gdb.arch/aarch64-sighandler-regs.c: New test. diff --git a/gdb/testsuite/gdb.arch/i386-avx512.exp b/gdb/testsuite/gdb.arch/i386-avx512.exp index d806d5f9a94..cd15e05fd03 100644 --- a/gdb/testsuite/gdb.arch/i386-avx512.exp +++ b/gdb/testsuite/gdb.arch/i386-avx512.exp @@ -93,9 +93,9 @@ gdb_test "break [gdb_get_line_number "third breakpoint here"]" \ gdb_continue_to_breakpoint "continue to third breakpoint in main" if [is_amd64_regs_target] { - set nr_regs 8 -} else { set nr_regs 32 +} else { + set nr_regs 8 } for { set r 0 } { $r < $nr_regs } { incr r } { -- 2.30.2