[llvm] Extend `retcon.once` coroutines lowering to optionally produce a normal result (PR #66333)

Chuanqi Xu via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 18:49:30 PDT 2023

@@ -3046,7 +3046,8 @@ void coro::buildCoroutineFrame(
   // Collect the spills for arguments and other not-materializable values.
   for (Argument &A : F.args())
     for (User *U : A.users())
-      if (Checker.isDefinitionAcrossSuspend(A, U))
+      if (Checker.isDefinitionAcrossSuspend(A, U) ||
+          isa<CoroEndInst>(U))
ChuanqiXu9 wrote:

Thanks. The API breaking is relatively acceptable since generally we don't have requirement for backport compatibility. 


More information about the llvm-commits mailing list