[PATCH] D48236: [IR] move shuffle mask queries from TTI to ShuffleVectorInst

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: lib/IR/Instructions.cpp:1691
+  bool UsesRHS = false;
+  for (int i = 0, NumElts = Mask.size(); i < NumElts; ++i) {
+    if (Mask[i] == -1)
----------------
lebedev.ri wrote:
> Oh, you can define multiple variables in init of the `for` loop?
> Somehow i never realized that..
Yep - http://llvm.org/docs/CodingStandards.html#don-t-evaluate-end-every-time-through-a-loop


https://reviews.llvm.org/D48236