[llvm] [Coroutines][LazyCallGraph] resumes are not really SCC (PR #116285)

[Tyler Nowicki via llvm-commits]

================
@@ -1720,6 +1720,7 @@ void LazyCallGraph::addSplitRefRecursiveFunctions(
   for (Function *NewFunction : NewFunctions) {
     Node &NewN = initNode(*NewFunction);
 
+    // Make the original function reference each new function
----------------
TylerNowicki wrote:

Can we just have addSplitRefRecursiveFunctions() call updateCGAndAnalysisManagerForCGSCCPass() then? Otherwise, it feels like we are just reinventing the function to identify the SCCs and RefSCCs in addSplitRefRecursiveFunctions(). The alternative is before adding the new functions to the CallGraph I add unreachable blocks to create 'fake' references between the new functions on an unused phi node to give them the appearance of being a RefSCC. Then delete those and then call updateCGAndAnalysisManagerForCGSCCPass().

https://github.com/llvm/llvm-project/pull/116285