[PATCH] D129636: Fix a LSR debug invariance issue

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 05:59:09 PDT 2022


jmorse added a comment.

Thanks for hunting this down, caching layers affecting codegen sounds hard to debug.

I'm no SCEV expert, so a couple of possibly obvious questions:

- Is it safe to store the SCEV* for a Value that's then forgotten?
- Is there a risk that the presence of debug-info will now cause a SCEV to be forgotten that _should_ have been remembered, because of unconditionally forgetting a SCEV if it's used by debug-info?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129636



More information about the llvm-commits mailing list