[PATCH] D110385: [SCEV] Verify contents of cached Block Dispositions

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 24 01:56:01 PDT 2021


mkazantsev added inline comments.


================
Comment at: llvm/test/Transforms/LCSSA/pr44058.ll:6
 
+; FIXME: breaks due to SCEV verification failure. Cached deleted values.
+; REQUIRES: asserts
----------------
fhahn wrote:
> Looks like all changed tests fail verification due to caching deleted values. Are there any cases where the disposition of a value changes?
All the failures are due to dangling pointers. We don't have an API to somehow invalidate this cache, and I'm surprised it doesn't break on every step. I've tried to fix things up in D110390.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110385/new/

https://reviews.llvm.org/D110385



More information about the llvm-commits mailing list