maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Clang] Fix long double availability check 

fae0dfa changed code to check 128-bit float availability, since it
introduced a new 128-bit double type on PowerPC. However, there're other
long float types besides IEEE float128 and PPC double-double requiring
this feature.

Reviewed By: ronlieb

Differential Revision: https://reviews.llvm.org/D109943
This commit is contained in:
Qiu Chaofan 2021-09-17 15:24:06 +08:00
parent 97cc678cc4
commit 0195f8621f
2 changed files with 31 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/Sema/Sema.cpp
View File

@ -1892,8 +1892,10 @@ void Sema::checkDeviceDecl(ValueDecl *D, SourceLocation Loc) {
bool LongDoubleMismatched = false;
if (Ty->isRealFloatingType() && Context.getTypeSize(Ty) == 128) {
const llvm::fltSemantics &Sem = Context.getFloatTypeSemantics(Ty);
if (!Ty->isIbm128Type() && !Ty->isFloat128Type() &&
&Sem != &Context.getTargetInfo().getLongDoubleFormat())
if ((&Sem != &llvm::APFloat::PPCDoubleDouble() &&
!Context.getTargetInfo().hasFloat128Type()) ||
(&Sem == &llvm::APFloat::PPCDoubleDouble() &&
!Context.getTargetInfo().hasIbm128Type()))
LongDoubleMismatched = true;
}

27
clang/test/OpenMP/amdgcn_ldbl_check.cpp Normal file
View File

@ -0,0 +1,27 @@
// REQUIRES: amdgpu-registered-target
// RUN: %clang_cc1 -triple x86_64-mingw64 -emit-llvm-bc -target-cpu x86-64 -fopenmp -fopenmp-targets=amdgcn-amd-amdhsa -o %t.bc -x c++ %s
// RUN: %clang_cc1 -triple amdgcn-amd-amdhsa -aux-triple x86_64-mingw64 -fsyntax-only -target-cpu gfx900 -fopenmp -fopenmp-is-device -fopenmp-host-ir-file-path %t.bc -x c++ %s
// expected-no-diagnostics
void print(double);
constexpr double operator"" _X (long double a)
{
return (double)a;
}
int main()
{
auto a = 1._X;
print(a);
#pragma omp target map(tofrom: a)
{
#pragma omp teams num_teams(1) thread_limit(4)
{
a += 1._X;
}
}
print(a);
return 0;
}