[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