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

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 07:45:13 PST 2017


samparker added a comment.

Ok, thanks. In this case the AND does get combined with the load, but a node also needs to be fixed up and so introduces another masking AND. This AND is then later combined with another node, in the process I assume added to the worklist, and then later hits visitAND. visitAND tries to combine with a load, which it does, but it also performs the fix up again, beginning the process again. I'm going to add another test case to satisfy myself that this works.


https://reviews.llvm.org/D41177





More information about the llvm-commits mailing list