[PATCH] D16691: [InstCombine] simplify masked load intrinsics with constant masks

[Elena Demikhovsky via llvm-commits]

delena accepted this revision.
delena added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:907
@@ +906,3 @@
+  // case Intrinsic::masked_store:
+  // case Intrinsic::masked_gather:
+  // case Intrinsic::masked_scatter:
----------------
spatel wrote:
> Please correct me if I've misunderstood, but I think scatter and gather each have one degenerate folding opportunity that we can handle here:
> 1. A scatter with zero mask is a nop.
> 2. A gather with zero mask will return the passthru arg.
yes, for zero mask it can be optimized.


http://reviews.llvm.org/D16691