This is an archive of the discontinued LLVM Phabricator instance.

[openmp] [test] Fix prepending config.library_dir to LD_LIBRARY_PATH
ClosedPublic

Authored by mgorny on Jul 30 2022, 11:58 AM.

Details

Summary

Fix the LD_LIBRARY_PATH prepending order to make sure that
config.library_path ends up before any potentially-system directories
(e.g. config.hwloc_library_dir). This makes sure that we are testing
against the just-built openmp libraries rather than the version that is
already installed.

Also rename the function to prepend_* to make it clearer what it
actually does.

Bug #56821

Diff Detail

Event Timeline

mgorny created this revision.Jul 30 2022, 11:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 30 2022, 11:58 AM
mgorny requested review of this revision.Jul 30 2022, 11:58 AM

@jlpeyton, if this looks good, could you also approve backporting to 15.x in https://github.com/llvm/llvm-project/issues/56821 ?

jlpeyton accepted this revision.Aug 1 2022, 9:35 AM

LGTM

This revision is now accepted and ready to land.Aug 1 2022, 9:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 9:54 AM