[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:02:44 PDT 2018


craig.topper added inline comments.


================
Comment at: test/CodeGen/X86/pr32420.ll:16
+; CHECK-NEXT:    movl %eax, %edx
+; CHECK-NEXT:    shll $12, %edx
+; CHECK-NEXT:    sarw $12, %dx
----------------
craig.topper wrote:
> There's some oddly repeated code in here. It looks like we managed to duplicate the load.
Nevermind, we didn't replicate it we just failed to combine the two logic cones.


https://reviews.llvm.org/D45585





More information about the llvm-commits mailing list