[libcxx-commits] [PATCH] D148478: [libc++][PSTL] Add design docs

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sun Apr 16 13:24:13 PDT 2023


philnik created this revision.
philnik added reviewers: ldionne, Mordante.
Herald added a subscriber: arphaman.
Herald added a project: All.
philnik requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D148478

Files:
  libcxx/docs/DesignDocs/PSTL.rst
  libcxx/docs/index.rst


Index: libcxx/docs/index.rst
===================================================================
--- libcxx/docs/index.rst
+++ libcxx/docs/index.rst
@@ -179,6 +179,7 @@
    DesignDocs/FileTimeType
    DesignDocs/HeaderRemovalPolicy
    DesignDocs/NoexceptPolicy
+   DesignDocs/PSTL
    DesignDocs/ThreadingSupportAPI
    DesignDocs/UniquePtrTrivialAbi
    DesignDocs/UnspecifiedBehaviorRandomization
Index: libcxx/docs/DesignDocs/PSTL.rst
===================================================================
--- /dev/null
+++ libcxx/docs/DesignDocs/PSTL.rst
@@ -0,0 +1,20 @@
+================
+PSTL integration
+================
+
+The PSTL project is quite complex in it's current form and does not provide everything that libc++ requires, for example
+``_LIBCPP_HIDE_FROM_ABI`` or similar annotations and including granularized headers. Because of these drawbacks, we
+decided to adopt a modified PSTL in libc++. Specifically, the goals of the modified PSTL are
+
+- No ``<__pstl_algorithm>`` and similar glue headers -- instead, the implementation files are included directly in
+  ``<algorithm>`` and friends.
+- No ``<pstl/internal/algorithm_impl.h>`` and ``<pstl/internal/algorithm_fwd.h>`` headers and friends -- these contain
+  the implementation and forward declarations for internal functions respectively. The implementation lives inside
+  ``<__algorithm/pstl_any_of.h>`` and friends, and the forward declarations are not needed inside libc++.
+- No ``<pstl/internal/glue_algorithm_defs.h>`` and ``<pstl/internal/glue_algorithm_impl.h>`` headers and friends --
+  these contain the public API. It lives inside ``<__algorithm/pstl_any_of.h>`` and friends instead.
+- The headers implementing backends are kept with as few changes as possible to make it easier to keep the backends in
+  sync with the backends from the original PSTL.
+- The configuration headers ``__pstl_config_site.in`` and ``pstl_config.h`` are removed, and any required configuration
+  is done inside ``__config_site.in`` and ``__config`` respectively.
+- libc++-style tests for the public PSTL API


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148478.514049.patch
Type: text/x-patch
Size: 2082 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230416/d96f326a/attachment.bin>


More information about the libcxx-commits mailing list