[PATCH] D42407: [X86] Use ISD::SIGN_EXTEND instead of X86ISD::VSEXT for mask to xmm/ymm/zmm conversion

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 12:42:59 PST 2018


craig.topper added a comment.

I think SelectionDAG::getNode does the same combine and we can't qualify that with LegalOperations. I can look at removing that or limiting it to scalar types and relying on a later DAG combine. But that seems like unnecessary churn to avoid 2 isel patterns.


https://reviews.llvm.org/D42407





More information about the llvm-commits mailing list