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

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 19 13:23:19 PDT 2016


mkuper added a comment.

In https://reviews.llvm.org/D24625#545669, @mkuper wrote:

> Anyway, I'll try and see if it affects any of our tests.


I'm not seeing any change in LIT tests from adding the mask legality check for any shuffle - either here or in https://reviews.llvm.org/D24683.
I'd really prefer to make the smaller change here and remove the check for the zero case.


https://reviews.llvm.org/D24625





More information about the llvm-commits mailing list