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

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 14 14:58:18 PDT 2020


nikic planned changes to this revision.
nikic added a comment.

@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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82799





More information about the llvm-commits mailing list