From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14156 invoked by alias); 5 Nov 2018 23:10:14 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 14028 invoked by uid 89); 5 Nov 2018 23:10:14 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-23.5 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,SPF_PASS,UNSUBSCRIBE_BODY autolearn=ham version=3.3.2 spammy=x0, Hx-spam-relays-external:209.85.221.67, H*RU:209.85.221.67 X-HELO: mail-wr1-f67.google.com Received: from mail-wr1-f67.google.com (HELO mail-wr1-f67.google.com) (209.85.221.67) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 05 Nov 2018 23:10:12 +0000 Received: by mail-wr1-f67.google.com with SMTP id o15-v6so7752808wrv.4 for ; Mon, 05 Nov 2018 15:10:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :in-reply-to:references; bh=jPYTiZe7GvQBIotEb8xS1Tq30Wlww3l502A70pfjlbs=; b=aiC+g88EFR67hJLMOh64xZpPaR6BhoO7Bj5Q76n3E6WgnB9t4qYWoAL6nbxHKhjO1C 1CgOkGEjfD1Cm3+YeesBC+dBtpksxNzSA+VAKoHHLXYcWjfA1F4qs/B2+UqKo04Vrq0F 2+nPoTWOvTSiYP9zkLlap4HllFz6is6bGBmjAdQRTaBs/KnnCBM6NoUxjrAYwWq0+tit z1N8UXoqTifx2LOXEZUlFXCbRePaHJSKlJ0E0CPS3QNG5qqBiFtzzljB8ecl10t2c53l BbFId4AGGigg8NJS5Iu0zBrHYksbTHWxGX3Ga4rfUyfT4qE7MBPew+A5Cfo6aMj1u3UA gq7g== Return-Path: Received: from localhost (host81-148-252-35.range81-148.btcentralplus.com. [81.148.252.35]) by smtp.gmail.com with ESMTPSA id v9-v6sm7886759wrm.6.2018.11.05.15.10.08 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 05 Nov 2018 15:10:08 -0800 (PST) From: Andrew Burgess To: gdb-patches@sourceware.org Cc: jimw@sifive.com, palmer@sifive.com, jhb@FreeBSD.org, Andrew Burgess Subject: [PATCH 2/2] gdb/riscv: Update test to support targets without FP hardware Date: Mon, 05 Nov 2018 23:10:00 -0000 Message-Id: <14c76a92c6491376e3a3829a2316ca8311a50b28.1541459121.git.andrew.burgess@embecosm.com> In-Reply-To: References: In-Reply-To: References: X-IsSubscribed: yes X-SW-Source: 2018-11/txt/msg00054.txt.bz2 Update gdb.arch/riscv-reg-aliases.exp test to support targets without floating point registers. gdb/testsuite/ChangeLog: * gdb.arch/riscv-reg-aliases.exp: Handle targets without floating point hardware. --- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.arch/riscv-reg-aliases.exp | 24 ++++++++++++++++++------ 2 files changed, 23 insertions(+), 6 deletions(-) diff --git a/gdb/testsuite/gdb.arch/riscv-reg-aliases.exp b/gdb/testsuite/gdb.arch/riscv-reg-aliases.exp index b4a2c982897..746ba7dda5d 100644 --- a/gdb/testsuite/gdb.arch/riscv-reg-aliases.exp +++ b/gdb/testsuite/gdb.arch/riscv-reg-aliases.exp @@ -151,17 +151,23 @@ check_zero_register_value "after write to \$x0" # we should access the register using 'REG_NAME.float'. In the following we # figure out if the field name is needed or not by looking at how GDB prints # on register. +set skip_freg_tests 0 set freg_extension "INVALID" set message "check format of float registers" -gdb_test_multiple "p \$ft0" $message { - -re " = {float = \[^\r\n\]+}\r\n$gdb_prompt $" { - set freg_extension ".float" +gdb_test_multiple "info registers \$ft0" $message { + -re "Invalid register `ft0'\r\n$gdb_prompt $" { + set skip_freg_tests 1 + set freg_extension "NONE" pass $message } - -re " = \[^{}\r\n\]+\r\n$gdb_prompt $" { + -re "ft0 \+\[0-9\]\+.*\r\n$gdb_prompt $" { set freg_extension "" pass $message } + -re "ft0 \+\{float = .*\r\n$gdb_prompt $" { + set freg_extension ".float" + pass $message + } } gdb_assert ![string eq "${freg_extension}" "INVALID"] \ "check that floating point format has been understood" @@ -169,7 +175,10 @@ gdb_assert ![string eq "${freg_extension}" "INVALID"] \ # Now check that we can write zero, and read zero back to all of the integer # and floating point registers. check_setting_registers_to_zero ${xreg_names} "" -check_setting_registers_to_zero ${freg_names} ${freg_extension} + +if { ! $skip_freg_tests } { + check_setting_registers_to_zero ${freg_names} ${freg_extension} +} # Set each register in turn to a new value, and confirm that the new value can # be read back from the primary name, and from all of the alias names. The @@ -177,4 +186,7 @@ check_setting_registers_to_zero ${freg_names} ${freg_extension} # significantly different so that the float tests don't reuse value from the # integer tests. check_setting_registers_to_value ${xreg_names} "" 100 -check_setting_registers_to_value ${freg_names} ${freg_extension} 500 + +if { ! $skip_freg_tests } { + check_setting_registers_to_value ${freg_names} ${freg_extension} 500 +} -- 2.14.5