<div>--target=<triple></div><div><br></div><div id="protonmail_mobile_signature_block"><div>Sent from ProtonMail Mobile</div></div> <div><br></div><div><br></div>On Fri, Jun 7, 2019 at 2:01 PM, Anton Afanasyev via Phabricator <<a href="mailto:reviews@reviews.llvm.org" class="">reviews@reviews.llvm.org</a>> wrote:<blockquote class="protonmail_quote" type="cite">  anton-afanasyev marked an inline comment as done.<br>anton-afanasyev added a comment.<br><br>In D61914#1534508 <https://reviews.llvm.org/D61914#1534508>, @plotfi wrote:<br><br>> cfe/trunk/test/Driver/check-time-trace.cpp appears to fail on Darwin. Did you mean to pass the target explicitly ?<br><br><br>This commit is reverted now. How can I pass the target explicitly?<br><br><br><br>================<br>Comment at: cfe/trunk/test/Driver/check-time-trace.cpp:1<br>+// RUN: %clangxx -ftime-trace %s 2>&1 | grep "Time trace json-file dumped to" \<br>+// RUN:   | awk '{print $NF}' | xargs cat \<br>----------------<br>plotfi wrote:<br>> This test should probably have // REQUIRES: shell<br>Thanks! I haven't found `REQUIRES: shell` here: `https://llvm.org/docs/CommandGuide/FileCheck.html`. Does this directive mean the target OS must have utils like `awk` and `xargs`?<br><br><br>Repository:<br>  rL LLVM<br><br>CHANGES SINCE LAST ACTION<br>  https://reviews.llvm.org/D61914/new/<br><br>https://reviews.llvm.org/D61914<br><br><br><br></blockquote><div><br></div><div><br></div>