[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