[PATCH] D52760: [SimplifyCFG] Change recursive calls to llvm::SimplifyCFG to instead call the "run" function on the SimplifyCFGOpt object
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 2 06:29:26 PDT 2018
reames added a comment.
Instead of doing the simplification recursively, wouldn't it be better to just iterate externally? That is, what if we split run into an outer function which called an inner helper routine as long as that inner routine was making progress? Or is there a concern that SimplifyCFG does not stabilize to a fixed point?
https://reviews.llvm.org/D52760
More information about the llvm-commits
mailing list