[libcxx-commits] [PATCH] D148480: [libc++][PSTL] Remove current integration

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Apr 20 08:19:39 PDT 2023


ldionne added a comment.

In D148480#4274264 <https://reviews.llvm.org/D148480#4274264>, @philnik wrote:

> In D148480#4274182 <https://reviews.llvm.org/D148480#4274182>, @Mordante wrote:
>
>> How does this change affect existing users of PSTL?
>> Are we certain the new code lands before LLVM 17?
>> I'm not sure how well the current PSTL implementation works, but if this affects users there should be release notes. We can always remove them when a new implementation is a drop-in replacement.
>
> The PSTL is broken in trunk, so this shouldn't affect anybody (anymore than it being broken already does). It will most likely land before LLVM 17 since I'll be working on it in the next months during my internship.

Just to clarify further -- we don't expect there are any users of the PSTL within libc++ right now, since the integration was never fully completed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148480



More information about the libcxx-commits mailing list