[PATCH] D31944: [DAGCombiner] add (sext i1 X), 1 --> zext (not i1 X)

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

We can clean up after legalization either way, I guess.  I'm more concerned about the test coverage; passing i1 vectors as arguments to a function isn't really representative of how i1 vectors are used in practice.


https://reviews.llvm.org/D31944