[libcxx-commits] [PATCH] D109373: [libc++] Revert OpenBSD-related changes to the documentation

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Sep 7 10:08:27 PDT 2021


ldionne created this revision.
Herald added subscribers: arphaman, krytarowski.
ldionne requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

This commit partially reverts 0954e2b2d038 <https://reviews.llvm.org/rG0954e2b2d0383584ee6ebad24e0637b98999e993> and 3fa4cff97480 <https://reviews.llvm.org/rG3fa4cff97480e47fad77e54b287061c492ab9ea8>, which
make changes to the libc++ documentation implifying that OpenBSD is
supported. Neither of these changes have been reviewed AFAICT, so
I'm reverting as a matter of enforcing:

1. That changes get reviewed before being committed
2. That we have a discussion and a support plan for supporting OpenBSD officially in libc++

Please note that I would be thrilled to support OpenBSD officially in
libc++, however doing so requires more than adding a note in the docs.
In particular, please make sure you read the note in [1] about setting
up CI testing for OpenBSD.

[1]: https://libcxx.llvm.org/#platform-and-compiler-support


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D109373

Files:
  libcxx/docs/UsingLibcxx.rst
  libcxx/docs/index.rst


Index: libcxx/docs/index.rst
===================================================================
--- libcxx/docs/index.rst
+++ libcxx/docs/index.rst
@@ -115,7 +115,6 @@
 =============== ========================= ============================
 macOS 10.9+     i386, x86_64, arm64       Building the shared library itself requires targetting macOS 10.11+
 FreeBSD 10+     i386, x86_64, arm
-OpenBSD         i386, x86_64, arm, arm64
 Linux           i386, x86_64, arm, arm64
 Windows         x86_64
 =============== ========================= ============================
Index: libcxx/docs/UsingLibcxx.rst
===================================================================
--- libcxx/docs/UsingLibcxx.rst
+++ libcxx/docs/UsingLibcxx.rst
@@ -68,8 +68,8 @@
 
   $ clang++ -stdlib=libc++ test.cpp
 
-On systems where libc++ is the library in use by default such as macOS, FreeBSD
-and OpenBSD, this flag is not required.
+On systems where libc++ is the library in use by default such as macOS and FreeBSD,
+this flag is not required.
 
 
 .. _alternate libcxx:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109373.371113.patch
Type: text/x-patch
Size: 1055 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210907/6991211f/attachment.bin>


More information about the libcxx-commits mailing list