[libcxx-commits] [PATCH] D148478: [libc++][PSTL] Add design docs
Nikolas Klauser via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed May 3 15:26:12 PDT 2023
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG5c6be1d48c15: [libc++][PSTL] Add design docs (authored by philnik).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148478/new/
https://reviews.llvm.org/D148478
Files:
libcxx/docs/DesignDocs/PSTLIntegration.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/PSTLIntegration
DesignDocs/ThreadingSupportAPI
DesignDocs/UniquePtrTrivialAbi
DesignDocs/UnspecifiedBehaviorRandomization
Index: libcxx/docs/DesignDocs/PSTLIntegration.rst
===================================================================
--- /dev/null
+++ libcxx/docs/DesignDocs/PSTLIntegration.rst
@@ -0,0 +1,22 @@
+================
+PSTL integration
+================
+
+The PSTL (or Parallel STL) project is quite complex in its current form and does not provide everything that libc++
+requires, for example ``_LIBCPP_HIDE_FROM_ABI`` or similar annotations and including granularized headers. Furthermore,
+the PSTL provides various layers of indirection that make sense in a generic implementation of the parallel algorithms,
+but are unnecessarily complex in the context of a single standard library implementation. 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.519283.patch
Type: text/x-patch
Size: 2357 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230503/8cea60e9/attachment.bin>
More information about the libcxx-commits
mailing list