[PATCH] D34397: [SCEV] Make MulOpsInlineThreshold lower to avoid excessive compilation time

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 22:29:36 PDT 2017


sanjoy accepted this revision.
sanjoy added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D34397#786261, @mkazantsev wrote:

> In https://reviews.llvm.org/D34397#785264, @joerg wrote:
>
> > What about the old TODO item of extending powi to integer types and folding them together in the middle end?
>
>
> Anyways, even if we have a concept of powi in IR, it does not automatically mean that we have it in SCEV. It would be extremely good to have it, though, but it looks like a massive task that takes time.


+1

Adding new nodes to SCEV has a high cost, and needs to have good justification.


https://reviews.llvm.org/D34397





More information about the llvm-commits mailing list