[PATCH] D55373: [LSR] Generate formulae to enable more post-incs

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

Okay, thanks. We're also seeing some regressions, so I know I've got some tuning to do. Do you have any idea of the characteristics of your regressions? At the moment I'm thinking:

- That the costs that I've added here are overly simplistic, for one I think I need to add a setup cost.
- It's also probably not worth doing when we know that the loop iteration count is low.
- In the current state, we also see code size regressions whereas your previous work helps us reduce code size. It may mean that I'll need a different flag to enable this change, but it also maybe a symptom of the performance regressions.


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

https://reviews.llvm.org/D55373