Revert "The _Float16 type is supported on x86 systems with SSE2 enabled."

This reverts commit 6623c02d70.
The change seems to be breaking build of compiler-rt on Debian.
This commit is contained in:
Zahira Ammarguellat
2021-11-23 08:00:57 -05:00
parent 3ff4e5f2a4
commit fd759d42c9
9 changed files with 5 additions and 86 deletions

View File

@@ -673,7 +673,7 @@ targets pending ABI standardization:
* 64-bit ARM (AArch64)
* AMDGPU
* SPIR
* X86 (Available with feature SSE2 and up enabled)
* X86 (Only available under feature AVX512-FP16)
``_Float16`` will be supported on more targets as they define ABIs for it.

View File

@@ -187,7 +187,6 @@ X86 Support in Clang
--------------------
- Support for ``AVX512-FP16`` instructions has been added.
- Support for ``_Float16`` type has been added.
Arm and AArch64 Support in Clang
--------------------------------

View File

@@ -239,9 +239,9 @@ bool X86TargetInfo::handleTargetFeatures(std::vector<std::string> &Features,
HasAVX512ER = true;
} else if (Feature == "+avx512fp16") {
HasAVX512FP16 = true;
HasFloat16 = true;
} else if (Feature == "+avx512pf") {
HasAVX512PF = true;
HasLegalHalfType = true;
} else if (Feature == "+avx512dq") {
HasAVX512DQ = true;
} else if (Feature == "+avx512bitalg") {
@@ -369,8 +369,6 @@ bool X86TargetInfo::handleTargetFeatures(std::vector<std::string> &Features,
.Default(NoXOP);
XOPLevel = std::max(XOPLevel, XLevel);
}
// Turn on _float16 for x86 (feature sse2)
HasFloat16 = SSELevel >= SSE2;
// LLVM doesn't have a separate switch for fpmath, so only accept it if it
// matches the selected sse level.

View File

@@ -1,73 +0,0 @@
// RUN: %clang_cc1 -triple x86_64-unknown-unknown -emit-llvm \
// RUN: < %s | FileCheck %s --check-prefixes=CHECK
_Float16 add1(_Float16 a, _Float16 b) {
// CHECK-LABEL: define{{.*}} half @add1
// CHECK: alloca half
// CHECK: alloca half
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: load half, half*
// CHECK: load half, half* {{.*}}
// CHECK: fadd half {{.*}}, {{.*}}
// CHECK: ret half
return a + b;
}
_Float16 add2(_Float16 a, _Float16 b, _Float16 c) {
// CHECK-LABEL: define{{.*}} half @add2
// CHECK: alloca half
// CHECK: alloca half
// CHECK: alloca half
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: load half, half* {{.*}}
// CHECK: load half, half* {{.*}}
// CHECK: fadd half {{.*}}, {{.*}}
// CHECK: load half, half* {{.*}}
// CHECK: fadd half {{.*}}, {{.*}}
// CHECK: ret half
return a + b + c;
}
_Float16 sub(_Float16 a, _Float16 b) {
// CHECK-LABEL: define{{.*}} half @sub
// CHECK: alloca half
// CHECK: alloca half
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: load half, half*
// CHECK: load half, half* {{.*}}
// CHECK: fsub half {{.*}}, {{.*}}
// CHECK: ret half
return a - b;
}
_Float16 div(_Float16 a, _Float16 b) {
// CHECK-LABEL: define{{.*}} half @div
// CHECK: alloca half
// CHECK: alloca half
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: load half, half* {{.*}}
// CHECK: load half, half* {{.*}}
// CHECK: fdiv half {{.*}}, {{.*}}
// CHECK: ret half
return a / b;
}
_Float16 mul(_Float16 a, _Float16 b) {
// CHECK-LABEL: define{{.*}} half @mul
// CHECK: alloca half
// CHECK: alloca half
// CHECK: store half {{.*}}, half*
// CHECK: store half {{.*}}, half*
// CHECK: load half, half* {{.*}}
// CHECK: load half, half* {{.*}}
// CHECK: fmul half {{.*}}, {{.*}}
// CHECK: ret half
return a * b;
}

View File

@@ -1,7 +1,5 @@
// RUN: %clang_cc1 -triple x86_64-linux -emit-llvm -target-feature +avx512fp16 < %s | FileCheck %s --check-prefixes=CHECK,CHECK-C
// RUN: %clang_cc1 -triple x86_64-linux -emit-llvm < %s | FileCheck %s --check-prefixes=CHECK,CHECK-C
// RUN: %clang_cc1 -triple x86_64-linux -emit-llvm -target-feature +avx512fp16 -x c++ -std=c++11 < %s | FileCheck %s --check-prefixes=CHECK,CHECK-CPP
// RUN: %clang_cc1 -triple x86_64-linux -emit-llvm -x c++ -std=c++11 < %s | FileCheck %s --check-prefixes=CHECK,CHECK-CPP
struct half1 {
_Float16 a;

View File

@@ -1,5 +1,4 @@
// RUN: %clang_cc1 %s -O0 -emit-llvm -triple x86_64-unknown-unknown -target-feature +avx512fp16 -o - | FileCheck %s --check-prefix=X86
// RUN: %clang_cc1 %s -O0 -emit-llvm -triple x86_64-unknown-unknown -o - | FileCheck %s --check-prefix=X86
_Float16 _Complex add_half_rr(_Float16 a, _Float16 b) {
// X86-LABEL: @add_half_rr(

View File

@@ -1,9 +1,8 @@
// RUN: %clang_cc1 -fsyntax-only -verify -triple x86_64-linux-pc %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple x86_64-linux-pc %s
// RUN: %clang_cc1 -fsyntax-only -verify -triple x86_64-linux-pc -target-feature +avx512fp16 %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple spir-unknown-unknown %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple armv7a-linux-gnu %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple aarch64-linux-gnu %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple i386-pc-linux-gnu %s
#ifndef HAVE
// expected-error@+2{{_Float16 is not supported on this target}}

View File

@@ -6,7 +6,7 @@
long double ld;
double d;
_Float16 f16;
_Float16 f16; // x86-error {{_Float16 is not supported on this target}}
int main() {
ld = d; // x86-warning {{implicit conversion increases floating-point precision: 'double' to 'long double'}}

View File

@@ -1,8 +1,7 @@
// RUN: %clang_cc1 -fsyntax-only -verify -triple x86_64-linux-pc %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple x86_64-linux-pc %s
// RUN: %clang_cc1 -fsyntax-only -verify -triple spir-unknown-unknown %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple armv7a-linux-gnu %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple aarch64-linux-gnu %s -DHAVE
// RUN: %clang_cc1 -fsyntax-only -verify -triple i386-pc-linux-gnu %s
#ifdef HAVE
// expected-no-diagnostics