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

[Markus Deuling <deuling at de dot ibm dot 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