[PATCH] D29715: LVI: Add a per-value worklist limit to LazyValueInfo.

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 9 01:43:59 PST 2017


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





More information about the llvm-commits mailing list