This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP][libomp] do not try to dlopen libmemkind on macOS
ClosedPublic

Authored by danieljdouglas-deco on May 27 2022, 4:37 PM.

Details

Summary

libmemkind is only available for linux
calling dlopen here can also be problematic in a client app that fork'ed

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 27 2022, 4:37 PM
danieljdouglas-deco requested review of this revision.May 27 2022, 4:37 PM
Herald added a project: Restricted Project. · View Herald Transcript
jlpeyton accepted this revision.May 31 2022, 1:37 PM

LGTM.

For future patches, look at https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface for uploading diff with as much context as possible.

This revision is now accepted and ready to land.May 31 2022, 1:37 PM

Sorry, I should have asked this before, do you need me to commit this on your behalf?

Sorry, I should have asked this before, do you need me to commit this on your behalf?

Ahh, yes please. I don't believe I have the permissions to do that. Thanks for your help!

This revision was automatically updated to reflect the committed changes.