[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