[PATCH] D15330: Fix cycle in selection DAG introduced by extractelement legalization

[Ahmed Bougacha via llvm-commits]

ab added a comment.

The fix seems safe to me, though I'm a little worried about hasPredecessor().


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:1483
@@ +1482,3 @@
+      // we will make the index dependent on the load).
+      if (Idx.getNode()->hasPredecessor(ST))
+        continue;
----------------
How about using hasPredecessorHelper?


http://reviews.llvm.org/D15330