[PATCH] D55963: [pstl] Initial integration with LLVM's CMake

Fri Dec 21 08:02:43 PST 2018

ldionne accepted this revision.
ldionne marked an inline comment as done.
ldionne added a comment.
This revision is now accepted and ready to land.

@jfb is on vacation, going forward. This is really more for exposition than anything anyway, since this is to get some testing going.

================
Comment at: pstl/cmake/FindTBB.cmake:12

-# Firstly search for TBB in config mode (i.e. search for TBBConfig.cmake).
-find_package(TBB QUIET CONFIG)
----------------
MikeDvorskiy wrote:
> ldionne wrote:
> > This is required because otherwise they look for both the release and the Debug version of TBB, and we don't find anything if both are not provided. The rest of `FindTBB.cmake` handles this properly, so I'm not sure why they even try to defer to the `TBBConfig.cmake`.
> Yes, the changes may be accepted, at least  till the TBBConfig.cmake will not require both Release and Debug TBB.
Please consider chiming in here: https://github.com/01org/tbb/issues/113

Repository:
  rPSTL pstl

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

https://reviews.llvm.org/D55963