[PATCH] D45195: Add --check-library-dependency to maintain compatibility with other linkers

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 01:32:45 PDT 2018


peter.smith added a comment.

In https://reviews.llvm.org/D45195#1058915, @ruiu wrote:

> `--find-library-backreferences` might be indeed better than `--check-library-dependency`, but both options are perhaps too long as a Unix linker option? How about `--detect-backrefs`? Maybe just `--backref`?


I think --backref is a bit too abstract, from looking at the option in a command line I would have no idea what it actually did without looking at the help. I think --detect-backrefs is better as it at least hints that it is of a diagnostic nature. I've got a mild preference for adding library or lib so we know what backrefs are referring to, but if everyone else is happy with --detect-backrefs then go with that. As an aside, Looking at the length of options in ld.bfd --help I think it has long given up on short option names, particularly for infrequently used ones.


https://reviews.llvm.org/D45195





More information about the llvm-commits mailing list