[PATCH] D39604: [DAGCombine] Move AND nodes to multiple load leaves

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 5 06:04:46 PST 2017


niravd accepted this revision.
niravd added a comment.
This revision is now accepted and ready to land.

Modulo the extra NodeToMask lines in zero extend case, this LGTM. Thanks!



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3779
+      // operation later.
+      if (NodeToMask)
+        return false;
----------------
Looks like you missed removing this case.


https://reviews.llvm.org/D39604





More information about the llvm-commits mailing list