Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D137986 Diff 476496 clang/test/CodeGen/aix-builtin-mapping.c

Changeset View

Standalone View

View Options

clang/test/CodeGen/aix-builtin-mapping.c

  • This file was added.
// AIX library functions frexpl, ldexpl, and modfl are for 128-bit IBM
// 'long double' (i.e. __ibm128). Check that the compiler generates
// calls to the 'double' versions for corresponding builtin functions in
daltentyUnsubmitted
Done
ReplyInline Actions
// AIX library functions frexpl, ldexpl, and modfl are for 128-bit
- // 'long double'. Check that the compiler generates calls to the 'double'
+ // IBM 'long double' (i.e. __ibm128). Check that the compiler generates calls to the 'double'
// versions for correspoding builtin functions in 64-bit 'long double' mode.
daltenty:
xingxueAuthorUnsubmitted
Done
ReplyInline Actions

Modified as suggested, thanks!

xingxue: Modified as suggested, thanks!
// 64-bit 'long double' mode.
hubert.reinterpretcastUnsubmitted
Done
ReplyInline Actions
// 'long double'. Check that the compiler generates calls to the 'double'
- // versions for correspoding builtin functions in 64-bit 'long double' mode.
+ // versions for corresponding builtin functions in 64-bit 'long double' mode.
// RUN: %clang_cc1 -triple powerpc-ibm-aix -mlong-double-64 -S -o - %s | FileCheck %s -check-prefix=CHECK

Fix typo.

hubert.reinterpretcast: Fix typo.
xingxueAuthorUnsubmitted
Done
ReplyInline Actions

Fixed, thanks for catching it!

xingxue: Fixed, thanks for catching it!
// RUN: %clang_cc1 -triple powerpc-ibm-aix -mlong-double-64 -emit-llvm -o - %s | FileCheck -check-prefix=CHECK %s
// RUN: %clang_cc1 -triple powerpc64-ibm-aix -mlong-double-64 -emit-llvm -o - %s | FileCheck -check-prefix=CHECK %s
hubert.reinterpretcastUnsubmitted
Done
ReplyInline Actions

This should be an IR test. Checking assembly means actually needing the PowerPC target enabled in the LLVM configuration.

hubert.reinterpretcast: This should be an IR test. Checking assembly means actually needing the PowerPC target enabled…
xingxueAuthorUnsubmitted
Done
ReplyInline Actions

Changed to use IR, thanks!

xingxue: Changed to use IR, thanks!
int main()
{
int DummyInt;
long double DummyLongDouble;
long double returnValue;
returnValue = __builtin_modfl(1.0L, &DummyLongDouble);
returnValue = __builtin_frexpl(0.0L, &DummyInt);
returnValue = __builtin_ldexpl(1.0L, 1);
}
// CHECK: %call = call double @modf(double noundef 1.000000e+00, ptr noundef %DummyLongDouble) #3
// CHECK: %call1 = call double @frexp(double noundef 0.000000e+00, ptr noundef %DummyInt) #3
// CHECK: %call2 = call double @ldexp(double noundef 1.000000e+00, i32 noundef {{(signext )?}}1) #4