[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 16:39:18 PST 2018


craig.topper added a comment.

Looks like I forgot to 'svn add' the test when i applied the patch before commit.

This patch should not effect those tests, but I'll double check.

We are not converting >8-bit or/and/xor to bts/btr/btc under optsize yet. Just >32 bit.  Using bts/btr/btc will cripple our load folding ability so its not a clear win.


https://reviews.llvm.org/D43201





More information about the llvm-commits mailing list