This is an archive of the discontinued LLVM Phabricator instance.

[lld][COFF] Find libraries with relative paths. r=hans,maskray,mstorsjo,phosek,rnk
ClosedPublic

Authored by thieta on Jul 14 2023, 12:43 AM.

Details

Summary

This patch is spun out of https://reviews.llvm.org/D151188
and makes it possible for lld-link to find libraries with
relative paths. This will be used later to implement the
changes to autolinking runtimes explained in #63827

Diff Detail

Event Timeline

thieta created this revision.Jul 14 2023, 12:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2023, 12:43 AM
thieta requested review of this revision.Jul 14 2023, 12:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2023, 12:43 AM
thieta retitled this revision from [lld][COFF] Find libraries with relative paths. r=hans,maskray,mstorsjo,phosek,rnk to [lld][COFF] Find libraries with relative paths.Jul 14 2023, 12:44 AM
hans accepted this revision.Jul 14 2023, 2:10 AM

lgtm

lld/docs/ReleaseNotes.rst
40

Perhaps a short example could be added here to make it more clear what changed.

This revision is now accepted and ready to land.Jul 14 2023, 2:10 AM
thieta updated this revision to Diff 540332.Jul 14 2023, 2:22 AM
thieta retitled this revision from [lld][COFF] Find libraries with relative paths to [lld][COFF] Find libraries with relative paths. r=hans,maskray,mstorsjo,phosek,rnk.

Add example to release notes

thieta retitled this revision from [lld][COFF] Find libraries with relative paths. r=hans,maskray,mstorsjo,phosek,rnk to [lld][COFF] Find libraries with relative paths..Jul 14 2023, 2:22 AM
hans added inline comments.Jul 14 2023, 2:24 AM
lld/docs/ReleaseNotes.rst
41

Technically, relative/path/my.lib would already work if that's relative to the current working directory. What's new is that it will now also work relative to /libpaths.

thieta updated this revision to Diff 540335.Jul 14 2023, 2:31 AM
thieta retitled this revision from [lld][COFF] Find libraries with relative paths. to [lld][COFF] Find libraries with relative paths. r=hans,maskray,mstorsjo,phosek,rnk.

Clarify release note even further

mstorsjo accepted this revision.Jul 14 2023, 2:59 AM

LGTM, thanks!

This revision was landed with ongoing or failed builds.Jul 14 2023, 3:09 AM
This revision was automatically updated to reflect the committed changes.