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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 17 11:45:06 PDT 2021


efriedma added a comment.

Looking at clang a bit more, I think the only way you end up with noalias arguments at the moment is via "__restrict".  Which ends up looking basically like your testcase.  Probably not too important in practice.

Patch looks fine from my side, but I'd like to leave final approval to someone more familiar with the coroutine code.


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