[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