[PATCH] D34025: [SCEV] Teach SCEVExpander to expand BinPow

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 14:36:34 PDT 2017


sanjoy requested changes to this revision.
sanjoy added a comment.
This revision now requires changes to proceed.

Hi Maxim,

If I understand your motivation here correctly, I think we've already lost by creating a `SCEVMulExpr` with an exponentially large number of operands.  IMO the right fix would be to prevent creating such expressions in the first place.


https://reviews.llvm.org/D34025





More information about the llvm-commits mailing list