[PATCH] D93828: [CGSCC][Coroutine][NewPM] Properly support function splitting/outlining

[Xun Li via Phabricator via llvm-commits]

lxfind added inline comments.


================
Comment at: llvm/lib/Analysis/LazyCallGraph.cpp:1716
+    // There is no reference from the new function back to the original
+    // function, so the new node is in its own SCC/RefSCC.
+    NewRC = createRefSCC(*this);
----------------
Could NewN be able to reach OriginalN by calling other functions?
For example, before CoroSplit, say A is the coroutine. A calls B, and B calls A.
After split, A references A.resume, A.resume calls B, and B calls A. In this case, they will still be in the same RefSCC?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93828/new/

https://reviews.llvm.org/D93828