Re: [RFA]: gdb/testsuite/gdb.arch/altivec-regs.exp Broken testcase

[Markus Deuling <deuling@de.ibm.com>]

Hi Mark,

Mark Kettenis schrieb:
>> Date: Tue, 16 Jan 2007 07:48:18 +0100
>> From: Markus Deuling <deuling@de.ibm.com>
>>
>> Hi,
>>
>> I'm referring to http://sourceware.org/ml/gdb-patches/2007-01/msg00353.html.
>>
>> It seems that the output from "show endian" has changed since the last time someone took a 
>> look into it :-) So I added a "re" to the test.
>>
>> Do we need this branch or can I delete it ?
>>>>     -re "(The target endianness is set automatically .currently )(big|little)( endian.*)$gdb_prompt $" {
>>>>         pass "endianness"
>>>>        set endianness $expect_out(2,string)
>>>>     }
> 
> This worries me a bit.  So in the past we were able to determine the
> endianness automagically, but now we don't?  Do we understand why?
> Doesn't this mean that something is broken that worked before?
> 
>

I looked into arch-utils.c.  We still need that branch above. But there are also changes, so that
we need to recognize "_("The target is assumed to be big endian\n"));". 

The variable "endianess" should either contain "big" or be empty in that testcase. So the way it's now with that patch should work, I think.

Regards,
Markus

-- 
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com