[PATCH] D45862: [InstCombine] Remove decanonicalizing transforms of selects

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 26 18:10:14 PDT 2018


mkazantsev added a comment.

I am confused. Didn't we agree on that replacement of selects with bit-wise logic, if it's profitable, should be a part of backend's work, and here we want selects because they have a more clear semantics that can be used by other transforms, rather than teaching every single transform to recognize the bit magic as select? You are giving some examples where codegen definitely could do what you are showing.


https://reviews.llvm.org/D45862





More information about the llvm-commits mailing list