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

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/docs/ReleaseNotes.rst:40
 ------------
+- Localization support has been made optional. It's enabled by default and can
+  be disabled by building libc++ with ``-DLIBCXX_ENABLE_LOCALIZATION=OFF``.
----------------
ldionne wrote:
> We also added support for systems without a random device. It can be disabled with `-DLIBCXX_ENABLE_RANDOM_DEVICE=OFF`.
Thanks! I'll add this information.


================
Comment at: libcxx/docs/ReleaseNotes.rst:49-50
+  This makes libc++ more ADL-proof.
+- The libc++ implementation status pages have been overhauled and look much
+  better now.
+- More C++20 features have been implemented. Especially the implementation of
----------------
tschuett wrote:
> modernized? look much better now is like opinion?
Good point, I'll change the wording.


================
Comment at: libcxx/docs/ReleaseNotes.rst:52
+- More C++20 features have been implemented. Especially the implementation of
+  the various ``constexpr`` papers made a lot of progress. :doc:`Cxx2aStatus`
+  has the full overview of libc++'s C++20 implementation status.
----------------
tschuett wrote:
> significant?
I've been doubting that myself. But if felt a bit empty without being able to mention some concrete improvement. However I don't object against removing it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95248