[PATCH] D29715: LVI: Add a per-value worklist limit to LazyValueInfo.
Daniel Berlin via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 9 01:58:19 PST 2017
Sigh.
Yeah, this is from replacing std::stack, which defaults to deque, with
SmallVector.
The upside is we don't have to pop one at a time, but the downside is it
invalidates references.
Hopefully none others are hiding here.
On Thu, Feb 9, 2017 at 1:43 AM, Vitaly Buka via Phabricator <
reviews at reviews.llvm.org> wrote:
> vitalybuka added inline comments.
>
>
> ================
> Comment at: llvm/trunk/lib/Analysis/LazyValueInfo.cpp:696
> // More work needs to be done before revisiting.
> - assert(BlockValueStack.top() != e && "Stack should have been
> pushed!");
> + assert(BlockValueStack.back() != e && "Stack should have been
> pushed!");
> }
> ----------------
> MSAN reported invalid memory access which was fixed with
> https://reviews.llvm.org/rL294572
>
>
> Repository:
> rL LLVM
>
> https://reviews.llvm.org/D29715
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170209/4beeebd4/attachment.html>
More information about the llvm-commits
mailing list