[PATCH] D119136: [clang] Implement Change scope of lambda trailing-return-type
Corentin Jabot via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 1 00:56:07 PDT 2022
cor3ntin marked an inline comment as done.
cor3ntin added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:18242
+bool CheckCaptureUseBeforeLambdaQualifiers(Sema &S, VarDecl *Var,
+ SourceLocation ExprLoc,
----------------
ChuanqiXu wrote:
> If this is only used in current file, we should mark it as static or put in anonymous namespace. And I think this lacks a comment
Good catch, thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119136/new/
https://reviews.llvm.org/D119136
More information about the cfe-commits
mailing list