Page MenuHomePhabricator

[Driver] [NetBSD] Pass default library search paths to linker
Needs ReviewPublic

Authored by mgorny on Jan 18 2019, 1:38 PM.

Details

Summary

Pass library search paths to linker as -L options, similarly to how
other toolchain drivers do it. This fixes compatibility with vanilla
LLD.

Bug report: https://bugs.llvm.org/show_bug.cgi?id=33155

// This is simplified version of D33726

Diff Detail

Event Timeline

mgorny created this revision.Jan 18 2019, 1:38 PM

Gentle ping.

ruiu added a comment.Jan 30 2019, 9:25 AM

Looks good but I'm probably not the right person to approve a change to this file.

There was a long discussion between two NetBSD maintainers about this (both already in the reviewers list of this patch). I'm not sure if there is an existing thread that would be better to follow up on as opposed to starting a fresh thread.

I think the question was: should the path search logic be handled in the clang driver or in LLD itself. FWIW, I prefer that NetBSD follow the same design as every other platform here with the (somewhat C and C++ specific) search logic provided by the clang driver.