This is an archive of the discontinued LLVM Phabricator instance.

Rename path libpath in .linker-options.
ClosedPublic

Authored by ruiu on Jan 30 2018, 1:04 PM.

Details

Summary

"path" is too generic name for -L or --library-path because a lot of
linker options take paths as arguments. This change renames the option
to avoid confusion.

Diff Detail

Repository
rL LLVM

Event Timeline

ruiu created this revision.Jan 30 2018, 1:04 PM
compnerd accepted this revision.Jan 30 2018, 3:43 PM

Sure, I'm not really tied to the names. Furthermore, I think that we will most likely have to revisit them anyways when the frontend changes are made as a different set of people may also have opinions on the cfe side :-) (which is why I really was trying to push the conversion to the side).

This revision is now accepted and ready to land.Jan 30 2018, 3:43 PM
ruiu added a comment.Jan 30 2018, 3:51 PM

I want this file to be the de facto standard document about the .linker-options section until we formalize it in a real standard document. That's why I wanted to update this doc.

This revision was automatically updated to reflect the committed changes.