aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

Aside from these minor nits and the naming concerns, I think this LGTM.



================
Comment at: clang/docs/ReleaseNotes.rst:747-748
 - Implemented "Permitting static constexpr variables in constexpr functions" 
(`P2647R1 <https://wg21.link/P2647R1>_`).
 - Implemented `CWG2640 Allow more characters in an n-char sequence 
<https://wg21.link/CWG2640>_`.
+  This Change was applied to C++20 as a Defect Report.
+- Implemented `P2036R3: Change scope of lambda trailing-return-type 
<https://wg21.link/P2036R3>`_
----------------
This change looks like it needs to be made on the Clang 16 branch as it was a 
drive-by change here.


================
Comment at: clang/docs/ReleaseNotes.rst:749-752
+- Implemented `P2036R3: Change scope of lambda trailing-return-type 
<https://wg21.link/P2036R3>`_
+  and `P2579R0 Mitigation strategies for P2036 <https://wg21.link/P2579R0>`_.
+  This proposals modify how variables captured in lambdas can appear in 
trailing return type
+  expressions and how their types are deduced therein, in all C++ language 
versions.
----------------
This will probably need rebasing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124351

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to