[PATCH] D115222: [Coroutines] Remove unused coroutine builtin/intrinsics llvm.coro.param (NFC-ish)

Tue Dec 7 18:44:18 PST 2021

ChuanqiXu added a comment.

In D115222#3177269 <https://reviews.llvm.org/D115222#3177269>, @rjmccall wrote:

> Like a lot of the switched-resume lowering, this intrinsic is extremely tied to C++ semantics.  If C++ doesn't actually allow the optimization anymore, then I completely agree that we should go ahead and remove the intrinsic.  If it's allowed and we just haven't implemented it yet, then okay, it might be best to remove it, but an unused intrinsic that we're planning to start using soon-ish isn't really doing any harm.

Understood. I think it is not allowed and I would consult with experts in EWG to make sure.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115222