RFC: The free function llvm::SimplifyCFG isn't used by any in-tree code, can we kill it?

More precisely, its only used within its own implementation and via the pass. Wondering if we should just sink all the code into the pass rather than having a free function.


This is the simplifycfg utility function that takes a worklist of BBs? I was planning on using it to fix http://llvm.org/pr20300. The pass would identify unreachable resumes, convert them to unreachable, and ask simplifycfg to propagate that backwards. Is that the right approach, or do I have the wrong pass?

Unless having the free function adds a lot of complexity, please keep it. I don't have an immediate use for it right now, but it seems like something which might be useful in an out of tree pass in the future.