Page MenuHomePhabricator

[clang-tools-extra] Fix linking dylib for LLVMFrontendOpenMP
ClosedPublic

Authored by mgorny on Dec 18 2019, 1:17 PM.

Details

Summary

Use LLVM_LINK_COMPONENTS to link the FrontendOpenMP library
instead of passing it explicitly to LINK_LIBS. This fixes duplicating
the library when clang-tidy is linked to LLVM dylib.

Diff Detail

Event Timeline

mgorny created this revision.Dec 18 2019, 1:17 PM
lebedev.ri accepted this revision.Dec 18 2019, 1:27 PM

LG
I think this is correct solution.

This revision is now accepted and ready to land.Dec 18 2019, 1:27 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptDec 18 2019, 1:41 PM