<div dir="ltr">I'm about to send a change that will fix the regression test (it should be -march=x86). It should fix build failures like:<div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_llvm-2Ds390x-2Dlinux1_builds_16458&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=j50SKkRyLUKoXDC06axsfVy1z0lEWJpwN7ceBKXOTN0&e=">http://lab.llvm.org:8011/builders/llvm-s390x-linux1/builds/16458</a><br></div><div><br></div><div>However your is similar to that one:<br><div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_clang-2Dx86-5F64-2Dlinux-2Dselfhost-2Dabi-2Dtest_builds_5852&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=-P49MhDTRAAoBFxyku6HW2R0MV9d-9k1ai1euckEAoQ&e=">http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-abi-test/builds/5852</a><br></div><div>but I'm not sure what's wrong with that.</div><div><br></div><div>Give me 10 more minutes. If it will not help, please revert it.</div><div><br></div><div>- Paweł</div><div><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 22, 2015 at 10:47 PM Frédéric Riss <<a href="mailto:friss@apple.com">friss@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>This commit is by far the most likely culprit for the hangs we see on green dragon:</div><div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8080_green_job_clang-2Dstage2-2Dconfigure-2DRlto-5Fcheck_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=BjUi5lRIq1QHdDM6LbPYjHI3YWAuUbft0iJCsGUq7Y4&e=" target="_blank">http://lab.llvm.org:8080/green/job/clang-stage2-configure-Rlto_check/</a></div><div><br></div><div>Did you have any other failures? Would you mind reverting it to see if it fixes things? (This is after an LTO bootstrap, so it takes quite some time to reproduce/propagate).</div><div><br></div><div>Fred</div></div><div style="word-wrap:break-word"><br><div><blockquote type="cite"><div>On Jun 22, 2015, at 8:58 AM, Pawel Bylica <<a href="mailto:chfast@gmail.com" target="_blank">chfast@gmail.com</a>> wrote:</div><br><div><div>Author: chfast<br>Date: Mon Jun 22 10:58:11 2015<br>New Revision: 240291<br><br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D240291-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=zhIscUGS-4EEXWB_671MTJ_RGa4VSRb2uOHEAsGoCKc&e=" target="_blank">http://llvm.org/viewvc/llvm-project?rev=240291&view=rev</a><br>Log:<br>Fix shl folding in DAG combiner.<br><br>Summary: The code responsible for shl folding in the DAGCombiner was assuming incorrectly that all constants are less than 64 bits. This patch simply changes the way values are compared.<br><br>Test Plan: A regression test included.<br><br>Reviewers: andreadb<br><br>Reviewed By: andreadb<br><br>Subscribers: andreadb, test, llvm-commits<br><br>Differential Revision: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10602&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=ciQmz3Kkc2pniNJgLupW8V8xhLiBKfnWngu6ubLVBp8&e=" target="_blank">http://reviews.llvm.org/D10602</a><br><br>Added:<br>    llvm/trunk/test/CodeGen/X86/fold-vector-shl-crash.ll<br>Modified:<br>    llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br><br>Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_llvm_trunk_lib_CodeGen_SelectionDAG_DAGCombiner.cpp-3Frev-3D240291-26r1-3D240290-26r2-3D240291-26view-3Ddiff&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=7IxTnEkq7N_NzupM5D6lQdMmDqnFbJ-KhyCBFUUqKQA&e=" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=240291&r1=240290&r2=240291&view=diff</a><br>==============================================================================<br>--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)<br>+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Mon Jun 22 10:58:11 2015<br>@@ -4275,7 +4275,7 @@ SDValue DAGCombiner::visitSHL(SDNode *N)<br>   if (isNullConstant(N0))<br>     return N0;<br>   // fold (shl x, c >= size(x)) -> undef<br>-  if (N1C && N1C->getZExtValue() >= OpSizeInBits)<br>+  if (N1C && N1C->getAPIntValue().uge(OpSizeInBits))<br>     return DAG.getUNDEF(VT);<br>   // fold (shl x, 0) -> x<br>   if (N1C && N1C->isNullValue())<br><br>Added: llvm/trunk/test/CodeGen/X86/fold-vector-shl-crash.ll<br>URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_llvm_trunk_test_CodeGen_X86_fold-2Dvector-2Dshl-2Dcrash.ll-3Frev-3D240291-26view-3Dauto&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cny4i04PtnEW2n2y1YFool7bh8nSZNLl_DXs5apwKPM&s=GKZrjYJc4Vi_r2JmBdy4N_R_S4HN93JlADzsiEJhR-k&e=" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/fold-vector-shl-crash.ll?rev=240291&view=auto</a><br>==============================================================================<br>--- llvm/trunk/test/CodeGen/X86/fold-vector-shl-crash.ll (added)<br>+++ llvm/trunk/test/CodeGen/X86/fold-vector-shl-crash.ll Mon Jun 22 10:58:11 2015<br>@@ -0,0 +1,8 @@<br>+; RUN: llc < %s | FileCheck %s<br>+<br>+;CHECK-LABEL: test<br>+define <2 x i256> @test() {<br>+  %S = shufflevector <2 x i256> zeroinitializer, <2 x i256> <i256 -1, i256 -1>, <2 x i32> <i32 0, i32 2><br>+  %B = shl <2 x i256> %S, <i256 -1, i256 -1> ; DAG Combiner crashes here<br>+  ret <2 x i256> %B<br>+}<br><br><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br></div></div></blockquote></div><br></div></blockquote></div>