[libcxx-commits] [PATCH] D106770: [libc++][doc] Update the release notes

[Louis Dionne via Phabricator via libcxx-commits]

ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

Thanks for doing that!



================
Comment at: libcxx/docs/ReleaseNotes.rst:41
 
-- ...
+- Support for older compilers has been removed. Several additional platforms
+  are now officially supported. :ref:`platform_and_compiler_support` contains
----------------
This shouldn't be under "New features". Either move to `API Changes` or put directly under "What's new in Libc++".


================
Comment at: libcxx/docs/ReleaseNotes.rst:44
+  the complete overview of platforms and compilers supported by libc++.
+- The large headers ``<algorithm>``, ``<iterator>``, and ``<utility>`` have
+  been split in more granular headers. This reduces the size of included code
----------------
I wouldn't put this under "New Features" either.


================
Comment at: libcxx/docs/ReleaseNotes.rst:49
+- The implementation of the C++20 concepts library has been completed.
+- Work on C++20 ranges has been started. :doc:`Status/Ranges` has the full
+  overview of libc++'s C++20 ranges implementation status.
----------------
We have to talk about this, but I was thinking about *not* shipping our incomplete Ranges implementation in LLVM 13. I was thinking about waiting until we have a proper implementation and we can guarantee ABI stability with more confidence.

While we have a decent part of Ranges implemented, some key parts are still missing and I'm not sure we want to ship in that state - most users are just going to get confused about what works and what doesn't.

That wouldn't impact the concepts.

@zoecarver @cjdb 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106770