[PATCH] D115222: [Coroutines] Remove unused coroutine builtin/intrinsics llvm.coro.param (NFC-ish)
Chuanqi Xu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
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
More information about the llvm-commits
mailing list