Don't add -Bsymbolic by default on Android.
authorEvgeniy Stepanov <eugeni.stepanov@gmail.com>
Wed, 16 Jul 2014 08:46:35 +0000 (08:46 +0000)
committerEvgeniy Stepanov <eugeni.stepanov@gmail.com>
Wed, 16 Jul 2014 08:46:35 +0000 (08:46 +0000)
-Bsymbolic is not a platform requirement and should not
be added unconditionally.

llvm-svn: 213126

clang/lib/Driver/Tools.cpp
clang/test/Driver/linux-ld.c

index 93f6e3e..8b8da30 100644 (file)
@@ -7168,9 +7168,6 @@ void gnutools::Link::ConstructJob(Compilation &C, const JobAction &JA,
       CmdArgs.push_back("-static");
   } else if (Args.hasArg(options::OPT_shared)) {
     CmdArgs.push_back("-shared");
-    if (isAndroid) {
-      CmdArgs.push_back("-Bsymbolic");
-    }
   }
 
   if (ToolChain.getArch() == llvm::Triple::arm ||
index 873cbcc..6a47d08 100644 (file)
 // RUN:     -shared \
 // RUN:   | FileCheck --check-prefix=CHECK-ANDROID-SO %s
 // CHECK-ANDROID-SO: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
-// CHECK-ANDROID-SO: "-Bsymbolic"
+// CHECK-ANDROID-SO-NOT: "-Bsymbolic"
 // CHECK-ANDROID-SO: "{{.*}}{{/|\\\\}}crtbegin_so.o"
 // CHECK-ANDROID-SO: "-L[[SYSROOT]]/usr/lib"
 // CHECK-ANDROID-SO-NOT: "gcc_s"