[PATCH] D76452: Use LLD by default for Android.

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 16 13:56:42 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.

LGTM.

But hope @ruiu or @int3 can clarify that we can't get rid of the `__APPLE__` special case in:

  // lld/tools/lld/lld.cpp
  static Flavor parseProgname(StringRef progname) {
  #if __APPLE__
    // Use Darwin driver for "ld" on Darwin.
    if (progname == "ld")
      return Darwin;
  #endif
  
  #if LLVM_ON_UNIX
    // Use GNU driver for "ld" on other Unix-like system.
    if (progname == "ld")
      return Gnu;
  #endif


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76452





More information about the cfe-commits mailing list