<div dir="auto">Fair enough. Shell glob expansion failures exit nonzero and fail the test, which is why the rm is gated by an if. Find/exec should work without a bash dependency though.</div><div class="gmail_extra"><br><div class="gmail_quote">On Apr 19, 2017 8:10 PM, "Dean Michael Berris via Phabricator" <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dberris accepted this revision.<br>
dberris added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM with one nit.<br>
<br>
<br>
<br>
================<br>
Comment at: test/xray/TestCases/Linux/fdr-<wbr>thread-order.cc:2<br>
<div class="quoted-text"> // RUN: %clangxx_xray -g -std=c++11 %s -o %t<br>
-// RUN: XRAY_OPTIONS="patch_premain=<wbr>false xray_naive_log=false xray_logfile_base=fdr-thread-<wbr>order. xray_fdr_log=true verbosity=1" %run %t 2>&1 | FileCheck %s<br>
+// RUN: if compgen -G "fdr-thread-order.*" ; then rm fdr-thread-order.* ; fi<br>
+// RUN: XRAY_OPTIONS="patch_premain=<wbr>false xray_naive_log=false xray_logfile_base=fdr-thread-<wbr>order. xray_fdr_log=true verbosity=1 xray_fdr_log_func_duration_<wbr>threshold_us=0" %run %t 2>&1 | FileCheck %s<br>
</div>----------------<br>
I don't think we can make the assumption that bash is the shell, but unconditionally removing the files anyway would be fine at this point.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D32259" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32259</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>