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

[Nirav Dave via Phabricator via llvm-commits]

niravd added a comment.

So if the Load is the same size as the mask, we used insert an AND but we don't end up eliding it? That seems like a fence post error elsewhere and it may be worth checking that out.

In any case, this change looks good. LGTM. Next time reopen the old revision, and update it. That way we can see the diffs directly on phabricator.


https://reviews.llvm.org/D41177