[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