[Openmp-commits] [PATCH] D144419: Improvements to the OpenMP Windows build

Fri Mar 24 07:25:43 PDT 2023

hans added a comment.

In D144419#4219493 <https://reviews.llvm.org/D144419#4219493>, @mstorsjo wrote:

> In D144419#4219492 <https://reviews.llvm.org/D144419#4219492>, @hans wrote:
>
>> Also, as mentioned above, the Windows LLVM release currently can't build OpenMP, so as it currently stands LLVM 17 will not include the OpenMP runtime for Windows.
>
> The issue that the build tried to use an option that `llvm-lib` didn't support, should have been fixed though - `llvm-lib` has got support for the option now.

Oh, that's good.

> When building the Windows LLVM releases, do you use `llvm-lib` from the previous version to build OpenMP, or does it first bootstrap everything with MSVC, and then use `llvm-lib` from the current version when building e.g. OpenMP? In the former case, it wouldn't work (until you get a new enough version of `llvm-lib`), in the latter case it should work now again.

It's the latter. Thanks!

It seems we still need the problem you mentioned fixed for the first stage of the build though.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144419