<HTML><HEAD>
<META content="text/html; charset=utf-8" http-equiv=Content-Type></HEAD>
<BODY>
<DIV>
<DIV style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif">Adding Yi for his input.</DIV></DIV>
<DIV dir=ltr>
<HR>
<SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif; FONT-WEIGHT: bold">From: </SPAN><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif"><A href="mailto:wschmidt@linux.vnet.ibm.com">Bill Schmidt</A></SPAN><BR><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif; FONT-WEIGHT: bold">Sent: </SPAN><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif">22/07/2014 21:45</SPAN><BR><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif; FONT-WEIGHT: bold">To: </SPAN><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif"><A href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</A></SPAN><BR><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif; FONT-WEIGHT: bold">Subject: </SPAN><SPAN style="FONT-SIZE: 11pt; FONT-FAMILY: Calibri,sans-serif">[PATCH][test-suite] Use ctimeit rather than perf for PowerPC</SPAN><BR><BR></DIV>[PATCH][test-suite] Use ctimeit rather than perf for PowerPC<BR><BR>A relatively recent patch (r209797) changed RunSafely.sh to use the timeit.sh<BR>script, which uses perf when available in preference to ctimeit. This causes<BR>problems on some older PowerPC distros, where perf has been notoriously<BR>buggy. As a result, the entire test suite appears to fail. There<BR>occasionally appears to be some flaky behavior from perf on newer distros as<BR>well which is less well-understood, but again results in phantom failures.<BR>IBM has recommended use of operf and ocount for a while in preference to perf.<BR><BR>Therefore we'd like to opt out of using perf on PowerPC in this script, even<BR>when perf is available. As with targets where perf is unavailable, we would<BR>like to fall back to ctimeit instead. This patch checks "uname -m" for ppc64<BR>or ppc64le to do just that.<BR><BR>Is this ok to commit?<BR><BR>Thanks,<BR>Bill<BR><BR><BR>Index: tools/timeit.sh<BR>===================================================================<BR>--- tools/timeit.sh (revision 213677)<BR>+++ tools/timeit.sh (working copy)<BR>@@ -1,6 +1,14 @@<BR>#! /bin/bash<BR># A wrapper over perf to provide similar functionality to timeit.c<BR><BR>+# Fall back to ctimeit for PowerPC<BR>+TARGET=`uname -m`<BR>+<BR>+if [ \( $TARGET = "ppc64" \) -o \( $TARGET = "ppc64le" \) ]; then<BR>+ $(dirname $0)/ctimeit $@<BR>+ exit $?<BR>+fi<BR>+<BR>DEPENDS="perf"<BR><BR># Fallback to ctimeit if dependencies are not met<BR><BR><BR>_______________________________________________<BR>llvm-commits mailing list<BR>llvm-commits@cs.uiuc.edu<BR>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<BR></BODY></HTML>