[PATCH] D53706: [RecursionStackElimination]: Pass to eliminate recursions
Martin Elshuber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 4 08:09:40 PST 2018
marels added a comment.
I implemented in a way such that it should be correct work on all architectures. I just use the TargetTransformInfo for the heuristics. Everything else completely target independent.
So, fundamentally wrong I think it is unlikely. If they profit from converting recursion into loops by explicitly modelling the state on the stack, I cannot tell (especially regarding performance).
As I can benchmark only on AArch64 in a suitable way, I prefer to enable this patch only for AArch64 and leave other architectures out for this initial version.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53706/new/
https://reviews.llvm.org/D53706
More information about the llvm-commits
mailing list