This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Fix import library installation with MinGW
ClosedPublic

Authored by mati865 on Aug 25 2020, 10:24 AM.

Details

Summary

Somehow I missed when submitting https://reviews.llvm.org/D85210 to review even though it was present in original patch for MSYS2 package.
Without this change CMake tries to install libomp.a (which doesn't exist for shared build) instead of libomp.dll.a.

Should not affect MSVC at all: https://gitlab.kitware.com/cmake/cmake/-/blob/5e212cafba0f2266510841eea3f5c7b79fde9449/Modules/Platform/Windows.cmake#L16 but makes big difference

Diff Detail

Event Timeline

mati865 created this revision.Aug 25 2020, 10:24 AM
mati865 requested review of this revision.Aug 25 2020, 10:24 AM
This revision is now accepted and ready to land.Aug 26 2020, 10:15 AM

I don't have write permission, could somebody commit it for me?
Thanks in advance.