[PATCH] D43201: [X86] Only reorder srl/and on last DAG combiner run

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 08:22:28 PST 2018


craig.topper added a comment.

All valid points. I don't know of any benchmarks that care either. I guess my goal with this patch is to get DAG going into isel to be consistent. The test cases in test-shrink.ll look very similar, but some use test and some use bt. This patch gets them all back to using test. We can isel them back to bt for https://reviews.llvm.org/D37418.


https://reviews.llvm.org/D43201





More information about the llvm-commits mailing list