[PATCH] D144627: [Clang] Fix a crash when taking the address of a consteval lambda

2023-02-23 Thread Corentin Jabot via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG34abc5b75d9d: [Clang] Fix a crash when taking the address of a consteval lambda (authored by cor3ntin). Changed prior to commit: https://reviews.llvm.org/D144627?vs=499773&id=499831#toc Repository: r

[PATCH] D144627: [Clang] Fix a crash when taking the address of a consteval lambda

2023-02-23 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM with a minor cleanup. Comment at: clang/lib/Sema/SemaExpr.cpp:17976-17983 auto *FD = cast(DR->getDecl()); -SemaRef.Diag(DR->getBeginLoc(), diag::e

[PATCH] D144627: [Clang] Fix a crash when taking the address of a consteval lambda

2023-02-23 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin updated this revision to Diff 499773. cor3ntin added a comment. Release notes + formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144627/new/ https://reviews.llvm.org/D144627 Files: clang/docs/ReleaseNotes.rst clang/include/cla

[PATCH] D144627: [Clang] Fix a crash when taking the address of a consteval lambda

2023-02-23 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin created this revision. Herald added a project: All. cor3ntin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The `_invoke` function of lambdas was not respecting the constexpr/consteval specifier of the call operator, so it was pos