[PATCH] D115790: [Coroutines] Set presplit attribute in Clang
Chuanqi Xu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 16 03:03:29 PST 2021
ChuanqiXu added a comment.
@ezhulenev hi, we are discussing if we should set the coroutine attributes in middle-end or front-end. Now in this revision, it would set the `coroutine.presplit` attribute in frontend, which would break the MLIR test. I have tried to implement it myself today. But I found that it is really not easy to edit MLIR's code. I found that you introduced the Coroutines intrinsic into MLIR. How do you think about set the attribute in the frontend of MLIR? And if you approve this, could you help to add the attributes in MLIR? We could still add the attribute in CoroEarly pass for a short time to make things consistent.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115790/new/
https://reviews.llvm.org/D115790
More information about the llvm-commits
mailing list