[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