[llvm] r216307 - Revert r215611 because it caused the infinite loop in bug 20736. There is a reduced testcase in that bug.

Nick Lewycky nicholas at mxc.ca
Fri Aug 22 17:45:03 PDT 2014


Author: nicholas
Date: Fri Aug 22 19:45:03 2014
New Revision: 216307

URL: http://llvm.org/viewvc/llvm-project?rev=216307&view=rev
Log:
Revert r215611 because it caused the infinite loop in bug 20736. There is a reduced testcase in that bug.

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
    llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v16.ll

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp?rev=216307&r1=216306&r2=216307&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp Fri Aug 22 19:45:03 2014
@@ -156,8 +156,6 @@ public:
   // Node replacement helpers
   void ReplacedNode(SDNode *N) {
     LegalizedNodes.erase(N);
-    if (UpdatedNodes)
-      UpdatedNodes->insert(N);
   }
   void ReplaceNode(SDNode *Old, SDNode *New) {
     DEBUG(dbgs() << " ... replacing: "; Old->dump(&DAG);

Modified: llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v16.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v16.ll?rev=216307&r1=216306&r2=216307&view=diff
==============================================================================
--- llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v16.ll (original)
+++ llvm/trunk/test/CodeGen/X86/vector-shuffle-128-v16.ll Fri Aug 22 19:45:03 2014
@@ -1,6 +1,11 @@
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mcpu=x86-64 -x86-experimental-vector-shuffle-lowering | FileCheck %s --check-prefix=ALL --check-prefix=SSE2
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mcpu=x86-64 -mattr=+ssse3 -x86-experimental-vector-shuffle-lowering | FileCheck %s --check-prefix=ALL --check-prefix=SSSE3
 
+; XFAIL: *
+; Only @stress_test0 is expected to fail, but XFAIL is not that selective. I
+; expect this to be unxfailed soon enough that we won't regress the other tests
+; in the interim.
+
 target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
 target triple = "x86_64-unknown-unknown"
 





More information about the llvm-commits mailing list