This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: i386-cmov1.c broken


On Sat, 2003-06-07 at 10:52, Daniel Jacobowitz wrote:
> On Sat, Jun 07, 2003 at 07:36:29PM +0200, Jan Hubicka wrote:
> > > > > 
> > > > > In auditing the GCC tests, I think that gcc.dg/i386-cmov1.c is passing
> > > > > "by accident".
> > > > > 
> > > > > In particular, this test seems to be trying to verify that the "sar"
> > > > > instruction is being used with "magic_namec" by scanning the assembly
> > > > > file.
> > > > > 
> > > > > However, that's not actually happening.
> > > > > 
> > > > > The scan-assembler test:
> > > > > 
> > > > >   /* { dg-final { scan-assembler "sar.*magic_namec" } } */
> > > > > 
> > > > > which is trying to test for this is busted because "." in Tcl includes
> > > > > "\n", i.e., the newline character.
> > > > Oops, thi sis ugly.  Tranks for noticing that, Mark!
> > > > I will fix the i386-* tests tonight.
> > > ! /* { dg-final { scan-assembler "pand\[^\\n\]magic" } } */
> > > ! /* { dg-final { scan-assembler "pandn\[^\\n\]magic" } } */
> > Unforutnately this does not work.  Does anyone know of better way of
> > encoding this in TCL except for using the newline character?
> 
> You lost a * after the \] - does adding that fix it?

It should.  Jan, with that change the testsuite patches are fine; please
check it in.

-- 
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]