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

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 20 12:13:34 PDT 2022

cor3ntin added a comment.

In D119136#3462707 <https://reviews.llvm.org/D119136#3462707>, @amyk wrote:

> In D119136#3459738 <https://reviews.llvm.org/D119136#3459738>, @alanphipps wrote:
>> We've also been seeing failures in our downstream Arm compiler when running the Perennial C++14 compliance tests related to this change.  Specifically, the tests expect a diagnostic to be issued when the lambda capture variable is outside of the lambda's {} scope. Another tests uses the lambda capture variable in a decltype-style return type which is also outside of the scope.
>> Do these failures sound like what others have run into?
> Hi, I would just like to express that I am also seeing these two issues on Power, that are a result of the original patch (D119136 <https://reviews.llvm.org/D119136>) that @alanphipps has pointed out.

Do you have a code example, just to make sure we are talking about the same thing?

  [i = 0]() -> decltype(i) {}

For example now is supposed to be valid with this change. I'm not sure how conformance test suits handle back-ported language changes and how long it takes them to be updated.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list