[PATCH] D26556: [InstCombine] don't widen most selects by hoisting an extend

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 15:14:25 PST 2016

efriedma added a comment.

If we could pattern-match our way out, it might be okay... but I don't think that's realistic in more complicated cases.  The sign-extend could be pushed forward through another operation or land in a different basic block.  I think it makes more sense to just try to make selects use the "right" width for the target (the width of the compare operands for most targets).


More information about the llvm-commits mailing list