[PATCH] D28158: [SCEV] limit recursion depth and operands number in getAddExpr

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov added a comment.

> What do you mean exactly? getMulExpr does not have a depth parameter like this. We have a comparison depth limit and a depth limit on SimplifyICmpOperands.

The second point is about new AddOpsInlineThreshold parameter. It is almost the same as MulOpsInlineThreshold used in getMulExpr (line 2608). And this part of change is independent of recursion depth limit, but suggested for the same purpose - to reduce almost infinite time of processing very long expressions.


https://reviews.llvm.org/D28158