[PATCH] D157296: [AST][Coroutine] Fix CoyieldExpr missing end loc
Shafik Yaghmour via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 22 16:03:01 PDT 2023
shafik added inline comments.
================
Comment at: clang/lib/Sema/SemaCoroutine.cpp:322
+ auto EndLoc = Args.empty() ? Loc : Args.back()->getEndLoc();
+ return S.BuildCallExpr(nullptr, Result.get(), Loc, Args, EndLoc, nullptr);
}
----------------
We should use [bugprone-argument-comment](https://clang.llvm.org/extra/clang-tidy/checks/bugprone/argument-comment.html) when using literals for arguments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157296/new/
https://reviews.llvm.org/D157296
More information about the cfe-commits
mailing list