[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