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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Looks like a good thing to verify to me, thanks! I can't tell due to missing context in the diff, are the new checks enabled by default? If so, do you think this will cause any verification failures during bootstrapping that should be addressed first?



================
Comment at: llvm/test/Transforms/LCSSA/pr44058.ll:6
 
+; FIXME: breaks due to SCEV verification failure. Cached deleted values.
+; REQUIRES: asserts
----------------
Looks like all changed tests fail verification due to caching deleted values. Are there any cases where the disposition of a value changes?


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

https://reviews.llvm.org/D110385