[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