[PATCH] D24625: [DAG] Remove isVectorClearMaskLegal() check from vector_build dagcombine

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 17 09:47:58 PDT 2016


RKSimon added a comment.

I agree that the UsesZeroVector test shouldn't be there but we should probably use a TLI.isShuffleMaskLegal(Mask, InVT1) test and not just blindly combine to a shuffle.


https://reviews.llvm.org/D24625





More information about the llvm-commits mailing list