[PATCH] D127528: [Clang] Let the linker choose shared or static libunwind unless specified

[Stephen Hines via Phabricator via cfe-commits]

srhines added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:1477
   if (Args.hasArg(options::OPT_shared_libgcc))
     return LibGccType::SharedLibGcc;
-  // The Android NDK only provides libunwind.a, not libunwind.so.
----------------
phosek wrote:
> mstorsjo wrote:
> > Previously, `OPT_shared_libgcc` would override the default for android, but now it no longer would. I guess that might be sensible, but it seems slightly outside of what the patch says it does. (I guess it's fine to include the change too, but I just wanted to raise it for discussion.)
> I was going by the comment, but I'm fine preserving the existing behavior. @srhines do you know if `-shared-libgcc` should be supported on Android?
No, there's no such shared library to even depend on, so it wouldn't be supported previously or in the future.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127528