[PATCH] D34174: [x86] replace div/rem with shift/mask for shuffle combining

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

This makes lots of sense to me. Honestly, fixing the multiplies might be worth it -- if you can benchmark >1% improvement, I might do it. I'm thinking about the architectures where integer multiplies are *very* slow (atom at least, also some ARM chips)


https://reviews.llvm.org/D34174