[clang] [llvm] [Coroutines] Replace struct alloca frame with byte array and ptradd (PR #178359)

Nikita Popov via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 3 00:46:03 PST 2026


nikic wrote:

It's sometimes hard to read intent when it's not explicitly stated. In this case the PR was already approved, and the questions answered without further response, so I think it was fairly reasonable to assume that there aren't further concerns.

To avoid misunderstandings, especially for PRs that already have approvals, I'd recommend explicitly writing something like "Please wait for me to review this as well" to make it clear that the PR shouldn't be merged just yet.

https://github.com/llvm/llvm-project/pull/178359


More information about the cfe-commits mailing list