[PATCH] D82799: [IndVars] Delay forgetValue() call

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D82799#2151699 <https://reviews.llvm.org/D82799#2151699>, @nikic wrote:

> @fhahn Thanks for taking a look!
>
> For the first diff I checked, the issue appears to be lack of "ext of min/max" to "min/max of ext" canonicalization in SCEV. Which is a peculiar oversight, because SCEV already tries to push exts through most everything else. I'll try adding that.


Sounds good. I could also take a look tomorrow, if you could share a test case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82799