[PATCH] D54899: [LoopStrengthReduce] ComplexityLimit as an option

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.

Hi @samparker,

> [...]
>  With this form, the first three loads can be base + immediate offset, while the last can perform the post increment to create %base.iv.next. What do you think?

Do you see the formula you are interested in the considered solutions when you increase the limit?
If yes, I would check why this one is preferred against the one you want and see how to improve the cost model.
If not, we'll have to teach LSR how to produce this formula in the generateXXX methods.

Cheers,
-Quentin


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54899