[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.


More information about the llvm-commits mailing list