[PATCH] D108006: [lld][ELF] Add --no-search-static-libs-for-shlib-undefined flag

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

I want to hear what Peter thinks of this.

https://sourceware.org/pipermail/binutils/2021-August/117696.html has an argument "would expect libfoo.so undefined symbols to similarly cause elements to be extracted from libbar.a.  Anything else is surprising."
I can refute that with https://maskray.me/blog/2021-08-15-extract-archive-member-to-satisfy-dso-undef#did-dynamic-linking-should-be-similar-to-static-linking-affect-the-design

However, https://groups.google.com/g/generic-abi/c/NyH6f470Cuc has an argument (somewhat convincing to me) that "Indirect dependencies (dependencies of dependencies) don't count" which makes sense to me,
so I am still struggling whether this really belongs to the upstream...

For the Android problem, it looks like a workaround as well. Do you know why gold linked executables were fine when Android was still using gold?
That "working fine" may be a gold bug.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108006/new/

https://reviews.llvm.org/D108006