[PATCH] D22904: Fix two bugs for musl-libc on ARM

Renato Golin via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 28 08:28:58 PDT 2016


rengolin added a comment.

Hi Lei,

Thanks for the fix. A few comments. Also, please, add some tests.

cheers,
--renato


================
Comment at: lib/Driver/ToolChains.cpp:4266
@@ -4266,1 +4265,3 @@
+  if (Triple.isAndroid()) {
     return Triple.isArch64Bit() ? "/system/bin/linker64" : "/system/bin/linker";
+  } else if (Triple.isMusl()) {
----------------
This is an early return, no need for brackets not the "else" below. Ex:

    if (Android)
      return ...
    
    if (musl) {
      ...
      return ...
    }


https://reviews.llvm.org/D22904





More information about the cfe-commits mailing list