Add NaCl to long double/fp128 mangling test

[dschuff]

Summary

NaCl is a platform where long double is the same as double.
Its mangling is spelled with "long double" but its ABI lowering is the same
as double.

[dschuff]

I'm not an expert in mangling, and I'm not 100% sure if this is right. The Itanium ABI's spec lists "long double" as the same as __float80. Should we use "double" in mangling instead?

[rnk]

lgtm

Feel free to commit more test coverage like this with just post-commit review.

[rnk]

In D11922#221185, @dschuff wrote:

I'm not an expert in mangling, and I'm not 100% sure if this is right. The Itanium ABI's spec lists "long double" as the same as __float80. Should we use "double" in mangling instead?

If you can have different overloads between double and long double, we need a separate mangling.

[dschuff]

Looks like all the other unix targets that use IEEEDouble as their format do this too, including all the NaCl compilers.

[chh]

If there were only two different types,
we should have mangled name 'd' for double and 'e' for long double,
even if their implementations are the same.

The problem came when g++ has float80 and float128 types,
and long double could be implemented as float80 or float128.
Then 'e' is used for float80, 'g' for float128,
and 'long double' has mangled name 'e' or 'g', depending on whether
it is float80 or float128.

Yes, so far I know only Android and powerpc use fp128 for long double on x86_64.

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
+// RUN: %clang_cc1 -triple x86_64-nacl -emit-llvm -o - %s \
+// RUN:    | FileCheck %s --check-prefix=N64
 
 // 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 }*
+// N64: define void @_Z4testlfdeCe(i32, float, double, double, double {{.*}}, double