Correct x86_64 Android fp128 mangled name

[chh]

Summary

These changes are for Android x86_64 targets to be compatible with current Android g++.

https://llvm.org/bugs/show_bug.cgi?id=23897

• Use 'g' and 'Cg' for "long double" and "long double _Complex" mangled type names.

[rnk]

test/CodeGen/x86-long-double.cpp

1–12 ↗

(On Diff #30501)

Don't use %clang in CodeGen tests.

[majnemer]

test/CodeGen/x86-long-double.cpp

5–6 ↗

(On Diff #30501)

This test is called x86-long-double.cpp but you have powerpc tests here.

[chh]

Use %clang_cc1 in unit test.

[chh]

Change unit test name to long_double_fp128.

[rnk]

Thanks, this seems almost done. Sorry for missing the -O on the first round.

test/CodeGen/long_double_fp128.cpp

1 ↗

(On Diff #30504)

I don't think these tests need optimizations (-O).

[echristo]

test/CodeGen/long_double_fp128.cpp

1 ↗

(On Diff #30504)

If they do there's a different problem :)

[chh]

Remove -O from unit tests.

[rnk]

lgtm

[chh]

Thanks for the review and approval.

I will commit this patch when I get my commit access request approved.
If anyone can help me commit this patch, please go ahead.
Thank you.

cfe/trunk/lib/Basic/Targets.cpp

 namespace {
 // x86_64 Android target
 class AndroidX86_64TargetInfo : public LinuxTargetInfo<X86_64TargetInfo> {
 public:
   AndroidX86_64TargetInfo(const llvm::Triple &Triple)
       : LinuxTargetInfo<X86_64TargetInfo>(Triple) {
     LongDoubleFormat = &llvm::APFloat::IEEEquad;
   }
+
+  bool useFloat128ManglingForLongDouble() const override {
+    return true;
+  }
 };
 } // end anonymous namespace
 
 
 //===----------------------------------------------------------------------===//
 // Driver code
 //===----------------------------------------------------------------------===//
 

cfe/trunk/test/CodeGen/long_double_fp128.cpp

+// RUN: %clang_cc1 -triple x86_64-linux-android -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=A64
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=G64
+// RUN: %clang_cc1 -triple powerpc64-linux-gnu -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=P64
+// RUN: %clang_cc1 -triple i686-linux-android -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=A32
+// RUN: %clang_cc1 -triple i686-linux-gnu -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=G32
+// RUN: %clang_cc1 -triple powerpc-linux-gnu -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=P32
+
+// Check mangled name of long double.
+// Android's gcc and llvm use fp128 for long double.
+void test(long, float, double, long double, long double _Complex) { }
+// A64:  define void @_Z4testlfdgCg(i64, float, double, fp128, { fp128, fp128 }*
+// G64:  define void @_Z4testlfdeCe(i64, float, double, x86_fp80, { x86_fp80, x86_fp80 }*
+// P64:  define void @_Z4testlfdgCg(i64, float, double, ppc_fp128, ppc_fp128 {{.*}}, ppc_fp128
+// A32:  define void @_Z4testlfdeCe(i32, float, double, double, { double, double }*
+// G32:  define void @_Z4testlfdeCe(i32, float, double, x86_fp80, { x86_fp80, x86_fp80 }*
+// P32:  define void @_Z4testlfdgCg(i32, float, double, ppc_fp128, { ppc_fp128, ppc_fp128 }*