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

Thu Nov 25 12:19:38 PST 2021

fhahn added a comment.

In D114183#3154540 <https://reviews.llvm.org/D114183#3154540>, @ronlieb wrote:

> Both of our amdgpu buildbots are broken by this commit, when I revert it locally my build will pass.
> https://lab.llvm.org/buildbot/#/builders/193
> https://lab.llvm.org/buildbot/#/builders/165
>
> https://lab.llvm.org/buildbot/#/builders/165/builds/10841
> https://lab.llvm.org/buildbot/#/builders/193/builds/2281

I recommitted the patch. for now and also submitted a fix for the build failure, which was caused by the return type of the future having been changed from `auto` to `std::future` bc41653a1f289a63a1472b7364437515c9659f77 <https://reviews.llvm.org/rGbc41653a1f289a63a1472b7364437515c9659f77>. Everything should build again now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114183