[PATCH] D125292: [Coroutines] Introduce "coro_readnone" operand bundles (2/3)

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 01:52:59 PDT 2022


ChuanqiXu added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroEarly.cpp:238-244
       case Intrinsic::coro_id_retcon:
       case Intrinsic::coro_id_retcon_once:
       case Intrinsic::coro_id_async:
         // TODO: Remove the line once we support it in the corresponding
         // frontend.
         F.addFnAttr(CORO_PRESPLIT_ATTR);
         break;
----------------
@rjmccall Could we remove the check now? If yes, we could add a check in the beginning of the function to return quickly for normal functions.


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

https://reviews.llvm.org/D125292



More information about the llvm-commits mailing list