[PATCH] D139858: [SCEVExpander] Increase "cheap" expansion budget

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

> There has been several reports that the current budget is just a tad bit too small

At the same time, we also regularly get complaints about exit values getting expanded where just reusing the final IV value would be cheaper, and raising the expansion limit is certainly going to make that situation worse.

Raising the expansion budget will improve some cases and regress others. For a heuristic cutoff, this is expected and fine. However, I would expect some kind of supporting data that the new value is indeed, on average, an improvement and not a regression.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139858