[PATCH] D114183: [ThreadPool] Support returning futures with results.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

I think this should be ready for another look now. Windows tests are passing now and the failure on Debian is unrelated to the patch I think.

Unfortunately I wasn't able to find a way to convert `std::packaged_task<ResTy()>` to `std::packaged_task<void()>` with MSVC. To work around that, I updated the code to manually create a promise and wrap the task function into another callable that calls the function and updates the promise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114183