<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>I went ahead and reverted the commit in r358459 to unblock the bots.</div><div><br></div><div>Cheers,</div><div>Alex</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 15 Apr 2019 at 13:02, Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Fangrui,<div><br></div><div>As Tim pointed out, it looks like your commit relies on features that broke this particular test on the darwin buildbots. This is the failure we're seeing:</div><div><br></div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:13px;white-space:pre-wrap">Command Output (stderr):
--
sed: 1: "y/\t/ /": transform strings are not the same length
FileCheck error: '-' is empty.
FileCheck command line:  /Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/FileCheck -strict-whitespace /Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/test/tools/llvm-objdump/X86/disassemble-align.s</span><br></div><div><br></div><div>Could you please take a look? Here's the full log: <a href="http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental/60093/consoleFull" target="_blank">http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental/60093/consoleFull</a></div><div><br></div><div>Please let me know if you need any help reproducing the failure.</div><div>Cheers,</div><div>Alex</div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 15 Apr 2019 at 12:25, Tim Northover via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Mon, 15 Apr 2019 at 14:30, Fangrui Song via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> +# RUN: llvm-objdump -d -print-imm-hex %t | sed 'y/\t/ /' | FileCheck -strict-whitespace %s<br>
<br>
This sed command looks like it relies on GNU specific features. Could<br>
you fix it or revert please?<br>
<br>
Tim.<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</blockquote></div>