[PATCH] D40718: [LSR] Complexity tradeoff in GenerateReassociationsImpl

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added a comment.

In https://reviews.llvm.org/D40718#964316, @mkazantsev wrote:

> The other way is limiting SCEV transformation not by depth only, but also by the size of operands list, if it makes sense.


I think your current approach is fine (penalize depth for high "breadth"), but it just needs to somehow happen within SCEV (so that "depth" remains a SCEV implementation detail).


https://reviews.llvm.org/D40718