Skip to content

PR for llvm/llvm-project#67779 #719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Oct 2, 2023
Merged

PR for llvm/llvm-project#67779 #719

merged 3 commits into from
Oct 2, 2023

Conversation

llvmbot
Copy link
Member

@llvmbot llvmbot commented Oct 2, 2023

mizvekov and others added 3 commits October 2, 2023 11:17
In order to reduce noise for a MR.

(cherry picked from commit a5e280bc6bda10607e0e7c864e4d23fac02d00aa)
…(#67856)

Also switch the test case to use -NEXT to strictly match all lines.

(cherry picked from commit 7d7d9e462a8983d71e01786c9ad61a976ff10e8a)
… path (#67857)

Before af744f0, the first entry
among the search paths was the empty string, indicating searching
in (or starting from) the current directory. After
af744f0, the toolchain/clang
specific lib directories were added at the head of the search path.

This would cause lookups of literal file names or relative paths
to match paths in the toolchain, if there are coincidental files
with similar names there, even if they would be find in the current
directory as well.

Change addClangLibSearchPaths to append to the list like all other
operations on searchPaths - but move the invocation of the
function to the right place in the sequence.

This fixes #67779.

(cherry picked from commit f906fd53b5ce12f07aec394ddf900c7f9f583405)
@mstorsjo
Copy link
Member

mstorsjo commented Oct 2, 2023

(Backport-approval-bot) @tru - I believe you're the one to approve this one as well.

@tru tru merged commit b338a28 into release/17.x Oct 2, 2023
@tru tru deleted the llvm-issue67779 branch October 2, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lld looks for libraries given via relative paths at wrong place (relative to system paths)
4 participants