[PATCH] [test-suite] Fix failure to capture stderr in *.out-simple's

[Daniel Sanders]

I'm experiencing the problem on my x86_64 Debian Jessie system while cross-compiling to Mips and running the resulting binaries under qemu. perf is run on the host system so I believe it affects everyone with a sufficiently new perf.

I haven't been able to check whether the same problem affects mips/mipsel Debian Jessie systems because the new perf isn't compatible with the kernels I currently have to use for these systems. The test-suite falls back on ctimeit for these systems.

> -----Original Message-----
> From: Tom Stellard [mailto:tom at stellard.net]
> Sent: 17 December 2014 20:13
> To: Daniel Sanders
> Cc: llvm-commits at cs.uiuc.edu
> Subject: Re: [PATCH] [test-suite] Fix failure to capture stderr in *.out-simple's
> 
> On Fri, Dec 12, 2014 at 10:39:48PM +0000, Daniel Sanders wrote:
> > A very late ping. I'd completely forgotten about this patch but I just ran into
> the problem again while testing LLVM 3.5.1 (cross-compilation from x86_64 to
> mips) with clean test-suite sources.
> 
> How critical is this fix for 3.5.1?  Will the test-suite fail on new Debian
> for everyone, or is it just a Mips issue?
> 
> -Tom
> 
> > ________________________________
> > From: Daniel Sanders
> > Sent: 15 August 2014 15:24
> > To: llvm-commits at cs.uiuc.edu
> > Subject: [PATCH] [test-suite] Fix failure to capture stderr in *.out-simple's
> >
> > Hi,
> >
> > I've just dist-upgraded one of my Debian machines and I'm now finding that a
> large number of tests are failing the reference check on that machine. All the
> missing lines appear in test.log so it seems that stderr is no longer redirected to
> the *.out-simple along with stdout. This patch fixes this by redirecting stderr to
> stdout on the perf command.
> >
> > Daniel Sanders
> > Leading Software Design Engineer, MIPS Processor IP
> > Imagination Technologies Limited
> > www.imgtec.com<http://www.imgtec.com/>
> >
> 
> > Index: tools/timeit.sh
> >
> ================================================================
> ===
> > --- tools/timeit.sh	(revision 213378)
> > +++ tools/timeit.sh	(working copy)
> > @@ -26,7 +26,7 @@
> >  	shift 2
> >  done
> >
> > -perf stat -o stats $@ < $INPUT > $OUTPUT
> > +perf stat -o stats $@ < $INPUT > $OUTPUT 2>&1
> >
> >  EXITCODE=$?
> >