From: Matt Arsenault Date: Fri, 15 Jul 2016 21:33:06 +0000 (+0000) Subject: AMDGPU: Remove legacy ldexp builtin X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=571b15e0389df587bdd76d73be438ffee7280e89;p=clang AMDGPU: Remove legacy ldexp builtin git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@275623 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Basic/BuiltinsAMDGPU.def b/include/clang/Basic/BuiltinsAMDGPU.def index 4656677e7c..ea63ea10f8 100644 --- a/include/clang/Basic/BuiltinsAMDGPU.def +++ b/include/clang/Basic/BuiltinsAMDGPU.def @@ -99,12 +99,5 @@ BUILTIN(__builtin_r600_read_tidig_z, "Ui", "nc") BUILTIN(__builtin_r600_recipsqrt_ieee, "dd", "nc") BUILTIN(__builtin_r600_recipsqrt_ieeef, "ff", "nc") -//===----------------------------------------------------------------------===// -// Legacy names with amdgpu prefix -//===----------------------------------------------------------------------===// - -BUILTIN(__builtin_amdgpu_ldexp, "ddi", "nc") -BUILTIN(__builtin_amdgpu_ldexpf, "ffi", "nc") - #undef BUILTIN #undef TARGET_BUILTIN diff --git a/lib/CodeGen/CGBuiltin.cpp b/lib/CodeGen/CGBuiltin.cpp index dbff809ab5..c74e53ea84 100644 --- a/lib/CodeGen/CGBuiltin.cpp +++ b/lib/CodeGen/CGBuiltin.cpp @@ -7690,14 +7690,6 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned BuiltinID, return emitRangedBuiltin(*this, Intrinsic::r600_read_tidig_y, 0, 1024); case AMDGPU::BI__builtin_r600_read_tidig_z: return emitRangedBuiltin(*this, Intrinsic::r600_read_tidig_z, 0, 1024); - - // Legacy amdgpu prefix - case AMDGPU::BI__builtin_amdgpu_ldexp: - case AMDGPU::BI__builtin_amdgpu_ldexpf: { - if (getTarget().getTriple().getArch() == Triple::amdgcn) - return emitFPIntBuiltin(*this, E, Intrinsic::amdgcn_ldexp); - return emitFPIntBuiltin(*this, E, Intrinsic::AMDGPU_ldexp); - } default: return nullptr; } diff --git a/test/CodeGenOpenCL/builtins-amdgcn.cl b/test/CodeGenOpenCL/builtins-amdgcn.cl index 51f1586607..6cac2a44bc 100644 --- a/test/CodeGenOpenCL/builtins-amdgcn.cl +++ b/test/CodeGenOpenCL/builtins-amdgcn.cl @@ -268,22 +268,6 @@ void test_read_exec(global ulong* out) { // CHECK: declare i64 @llvm.read_register.i64(metadata) #[[NOUNWIND_READONLY:[0-9]+]] -// Legacy intrinsics with AMDGPU prefix - -// CHECK-LABEL: @test_legacy_ldexp_f32 -// CHECK: call float @llvm.amdgcn.ldexp.f32 -void test_legacy_ldexp_f32(global float* out, float a, int b) -{ - *out = __builtin_amdgpu_ldexpf(a, b); -} - -// CHECK-LABEL: @test_legacy_ldexp_f64 -// CHECK: call double @llvm.amdgcn.ldexp.f64 -void test_legacy_ldexp_f64(global double* out, double a, int b) -{ - *out = __builtin_amdgpu_ldexp(a, b); -} - // CHECK-LABEL: @test_kernarg_segment_ptr // CHECK: call i8 addrspace(2)* @llvm.amdgcn.kernarg.segment.ptr() void test_kernarg_segment_ptr(__attribute__((address_space(2))) unsigned char ** out) diff --git a/test/CodeGenOpenCL/builtins-r600.cl b/test/CodeGenOpenCL/builtins-r600.cl index 0951ff8182..027a54a6bc 100644 --- a/test/CodeGenOpenCL/builtins-r600.cl +++ b/test/CodeGenOpenCL/builtins-r600.cl @@ -17,22 +17,6 @@ void test_recipsqrt_ieee_f64(global double* out, double a) } #endif -// CHECK-LABEL: @test_legacy_ldexp_f32 -// CHECK: call float @llvm.AMDGPU.ldexp.f32 -void test_legacy_ldexp_f32(global float* out, float a, int b) -{ - *out = __builtin_amdgpu_ldexpf(a, b); -} - -#if cl_khr_fp64 -// XCHECK-LABEL: @test_legacy_ldexp_f64 -// XCHECK: call double @llvm.AMDGPU.ldexp.f64 -void test_legacy_ldexp_f64(global double* out, double a, int b) -{ - *out = __builtin_amdgpu_ldexp(a, b); -} -#endif - // CHECK-LABEL: @test_implicitarg_ptr // CHECK: call i8 addrspace(7)* @llvm.r600.implicitarg.ptr() void test_implicitarg_ptr(__attribute__((address_space(7))) unsigned char ** out)