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

[Ed Maste via Phabricator via llvm-commits]

emaste added inline comments.


================
Comment at: lld/trunk/ELF/Options.td:333
+defm warn_backrefs: B<"warn-backrefs",
+    "Warn about backward symbol references to fetch archive members",
+    "Do not warn about backward symbol references to fetch archive members">;
----------------
I'm not sure what fetch archive members means in this context. What about "warn about undefined symbol references satisifed by earlier archive members" or something like that?


================
Comment at: lld/trunk/ELF/SymbolTable.cpp:292
+//
+// --warn-backrefs is an option to prevent an undefined reference from
+// fetching an archive member written earlier in the command line. It can be
----------------
Does the option prevent this case or (in the absence of `-fatal-warnings`) only warn about it?


Repository:
  rL LLVM

https://reviews.llvm.org/D45195