[libcxx-commits] [PATCH] D99836: [pstl] Implement OpenMP backend

Ruslan Arutyunyan via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Oct 14 09:36:02 PDT 2021


rarutyun added a comment.

In D99836#3064347 <https://reviews.llvm.org/D99836#3064347>, @ldionne wrote:

> Leaving the correctness of the backend to @MikeDvorskiy (thanks!), but apart from that this looks good to me.

Together with Mikhail we reviewed that patch and it looks good. The tests were launched on the (almost) latest code base in oneDPL CI (just several configurations, to be honest). Anyway, no issues observed.

Do you want us to commit this patch?

> Let's work on enabling the tests in the CI?

I believe it can be done separately if you don't have opposite opinion


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99836



More information about the libcxx-commits mailing list