This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Adjust gdb.arch/i386-sse-stack-align.exp print statement
- From: Sergio Durigan Junior <sergiodj at redhat dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: GDB Patches <gdb-patches at sourceware dot org>
- Date: Fri, 08 Dec 2017 11:28:47 -0500
- Subject: Re: [RFA] Adjust gdb.arch/i386-sse-stack-align.exp print statement
- Authentication-results: sourceware.org; auth=none
- References: <1499959929-29497-9-git-send-email-palves@redhat.com> <20171207233015.18656-1-sergiodj@redhat.com> <71f1cec5-d299-6b08-4189-84f35e18fc4c@redhat.com>
On Friday, December 08 2017, Pedro Alves wrote:
> On 12/07/2017 11:30 PM, Sergio Durigan Junior wrote:
>>
>> commit 46a4882b3c7d9ec981568b8b13a3c9c39c8f8e61
>> Author: Pedro Alves <palves@redhat.com>
>> Date: Mon Sep 4 20:21:15 2017 +0100
>>
>> Stop assuming no-debug-info variables have type int
>>
>> We now have to explicitly tell GDB the type of the non-debug-info
>> variable we want to print (by casting). This commit adjusts the
>> "print" statement on gdb.arch/i386-sse-stack-align.exp to do the
>> proper cast, fixing a failure that started to happen after the
>> mentioned commit.
>>
>> gdb/testsuite/ChangeLog:
>> 2017-12-07 Sergio Durigan Junior <sergiodj@redhat.com>
>>
>> * gdb.arch/i386-sse-stack-align.exp: Cast "print" variable to
>> "int".
>
> This is OK, except...
>
>> ---
>> gdb/testsuite/ChangeLog | 5 +++++
>> gdb/testsuite/gdb.arch/i386-sse-stack-align.exp | 2 +-
>> 2 files changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
>> index 53f114aa15..81d23d1bb1 100644
>> --- a/gdb/testsuite/ChangeLog
>> +++ b/gdb/testsuite/ChangeLog
>> @@ -1,3 +1,8 @@
>> +2017-12-07 Sergio Durigan Junior <sergiodj@redhat.com>
>> +
>> + * gdb.arch/i386-sse-stack-align.exp: Cast "print" variable to
>> + "int".
>> +
>> 2017-12-07 Keith Seitz <keiths@redhat.com>
>>
>> * gdb.cp/nested-types.cc: New file.
>> diff --git a/gdb/testsuite/gdb.arch/i386-sse-stack-align.exp b/gdb/testsuite/gdb.arch/i386-sse-stack-align.exp
>> index 47fb9f42d2..946f217ce1 100644
>> --- a/gdb/testsuite/gdb.arch/i386-sse-stack-align.exp
>> +++ b/gdb/testsuite/gdb.arch/i386-sse-stack-align.exp
>> @@ -43,7 +43,7 @@ if ![runto_main] then {
>>
>> set args ""
>> foreach i {0 1 2 3 4} {
>> - set test "print g$i ($args)"
>> + set test "print (int) g$i ($args)"
>
> ... this is printing/calling a function, not a variable:
>
> (gdb) print g0 ()
> 'g0' has unknown return type; cast the call to its declared return type
>
> So the git log and ChangeLog are inaccurate and should
> reference 7022349d5c86 ("Stop assuming no-debug-info functions
> return int") instead.
>
> OK with that fixed.
Thanks for the catch, I adjusted the ChangeLog entry and the commit
message accordingly.
Pushed:
1cd9a73b4280cb133b305ee31a0e87f114bd1be8
--
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/