[PATCH] D45585: [DAGCombiner][X86] When promoting loads don't use ZEXTLOAD even its legal

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 12 13:26:12 PDT 2018


craig.topper added a comment.

It seems DAG combine doesn't know these properties which accounts for some of the redundancies seen in the test output. We can either improve DAG combine or just trust that InstCombine would have prevented this in the first place

(A&B)&A -> A & B
(A^B)^A -> B


https://reviews.llvm.org/D45585





More information about the llvm-commits mailing list