[PATCH] D119136: [clang] Implement Change scope of lambda trailing-return-type

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 13 21:37:29 PDT 2022


rsmith added a comment.

Hey @cor3ntin, sorry, I reverted this again. It's breaking thread-safety  annotations on lambdas in a pretty severe way:

  Mutex m;
  auto lambda = [=]() EXCLUSIVE_LOCKS_REQUIRED(m) {...}

doesn't compile any more. Presumably we should be parsing the attribute in the enclosing context rather than in the half-way-inside-the-lambda context in which we parse parameters?


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