[PATCH] D104184: [Coroutine] Properly deal with byval and noalias parameters

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 16 19:47:46 PDT 2021


ChuanqiXu added inline comments.


================
Comment at: llvm/test/Transforms/Coroutines/coro-byval-param.ll:6
+; Function Attrs: noinline ssp uwtable mustprogress
+define %promise_type* @foo(%struct.A* nocapture readonly byval(%struct.A) align 8 %a1) #0 {
+entry:
----------------
Since this patch deals with 'byval' and 'noalias' arguments, it'd better to add a test case for 'noalias' arguments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104184



More information about the llvm-commits mailing list