[PATCH] D99489: [clang] [PR49736] [C++2b] Correctly reject lambdas with requires clause and no parameter list

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 29 08:21:37 PDT 2021


curdeius added inline comments.


================
Comment at: clang/lib/Parse/ParseExprCXX.cpp:1435
+
+    WarnIfHasCUDATargetAttr();
   } else if (Tok.isOneOf(tok::kw_mutable, tok::arrow, tok::kw___attribute,
----------------
aaron.ballman wrote:
> Rather than add this in both branches, I'd say to hoist it out of the branches and put it after the `else if` below (before the declaration of `ScopeFlags`).
True. Will do soon.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99489/new/

https://reviews.llvm.org/D99489



More information about the cfe-commits mailing list