[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