[PATCH] D17485: [x86, InstCombine] transform x86 AVX masked stores to LLVM intrinsics

[Simon Pilgrim via llvm-commits]

RKSimon added a comment.

Couple of minor queries.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:824
@@ +823,3 @@
+  return ConstantVector::get(BoolVec);
+}
+
----------------
Not a priority but can the blendv instcombines share this helper (or something very similar)?

================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:1609
@@ +1608,3 @@
+  // TODO: The AVX2 integer variants can go here too.
+    return simplifyX86MaskedStore(*II, *this);
+
----------------
If this returns nullptr shouldn't the case break? 


http://reviews.llvm.org/D17485