[PATCH] D27751: [LLVM] Use after move bug fixes

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 17 04:44:29 PST 2016


Prazek added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:27000
   while (Mask.size() > 1 && canWidenShuffleElements(Mask, WidenedMask)) {
+    // No use after move, WidenMask reinitialized in canWidenShuffleElements.
     Mask = std::move(WidenedMask);
----------------
mboehme wrote:
> Typo: Should be WidenedMask
btw, why it fires here? It is passed to canWidenShuffleElements by reference and documentation says that it assumes it is being reinitialized in this case.


https://reviews.llvm.org/D27751





More information about the llvm-commits mailing list