[llvm] r215209 - Add missing Interpreter intrinsic lowering for sin, cos and ceil

[Josh Klontz]

Author: jklontz
Date: Fri Aug  8 10:00:12 2014
New Revision: 215209

URL: http://llvm.org/viewvc/llvm-project?rev=215209&view=rev
Log:
Add missing Interpreter intrinsic lowering for sin, cos and ceil

Added:
    llvm/trunk/test/ExecutionEngine/Interpreter/
    llvm/trunk/test/ExecutionEngine/Interpreter/intrinsics.ll
    llvm/trunk/test/ExecutionEngine/Interpreter/lit.local.cfg
Modified:
    llvm/trunk/lib/CodeGen/IntrinsicLowering.cpp
    llvm/trunk/test/lit.site.cfg.in

Modified: llvm/trunk/lib/CodeGen/IntrinsicLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/IntrinsicLowering.cpp?rev=215209&r1=215208&r2=215209&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/IntrinsicLowering.cpp (original)
+++ llvm/trunk/lib/CodeGen/IntrinsicLowering.cpp Fri Aug  8 10:00:12 2014
@@ -528,6 +528,18 @@ void IntrinsicLowering::LowerIntrinsicCa
     ReplaceFPIntrinsicWithCall(CI, "powf", "pow", "powl");
     break;
   }
+  case Intrinsic::sin: {
+    ReplaceFPIntrinsicWithCall(CI, "sinf", "sin", "sinl");
+    break;
+  }
+  case Intrinsic::cos: {
+    ReplaceFPIntrinsicWithCall(CI, "cosf", "cos", "cosl");
+    break;
+  }
+  case Intrinsic::ceil: {
+    ReplaceFPIntrinsicWithCall(CI, "ceilf", "ceil", "ceill");
+    break;
+  }
   case Intrinsic::flt_rounds:
      // Lower to "round to the nearest"
      if (!CI->getType()->isVoidTy())

Added: llvm/trunk/test/ExecutionEngine/Interpreter/intrinsics.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/Interpreter/intrinsics.ll?rev=215209&view=auto
==============================================================================
--- llvm/trunk/test/ExecutionEngine/Interpreter/intrinsics.ll (added)
+++ llvm/trunk/test/ExecutionEngine/Interpreter/intrinsics.ll Fri Aug  8 10:00:12 2014
@@ -0,0 +1,19 @@
+; RUN: lli -O0 -force-interpreter < %s
+
+; libffi does not support fp128 so we don’t test it
+declare float  @llvm.sin.f32(float)
+declare double @llvm.sin.f64(double)
+declare float  @llvm.cos.f32(float)
+declare double @llvm.cos.f64(double)
+declare float  @llvm.ceil.f32(float)
+declare double @llvm.ceil.f64(double)
+
+define i32 @main() {
+  %sin32 = call float @llvm.sin.f32(float 0.000000e+00)
+  %sin64 = call double @llvm.sin.f64(double 0.000000e+00)
+  %cos32 = call float @llvm.cos.f32(float 0.000000e+00)
+  %cos64 = call double @llvm.cos.f64(double 0.000000e+00)
+  %ceil32 = call float @llvm.ceil.f32(float 0.000000e+00)
+  %ceil64 = call double @llvm.ceil.f64(double 0.000000e+00)
+  ret i32 0
+}

Added: llvm/trunk/test/ExecutionEngine/Interpreter/lit.local.cfg
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/Interpreter/lit.local.cfg?rev=215209&view=auto
==============================================================================
--- llvm/trunk/test/ExecutionEngine/Interpreter/lit.local.cfg (added)
+++ llvm/trunk/test/ExecutionEngine/Interpreter/lit.local.cfg Fri Aug  8 10:00:12 2014
@@ -0,0 +1,3 @@
+# These tests require foreign function calls
+if config.enable_ffi != "ON":
+    config.unsupported = True

Modified: llvm/trunk/test/lit.site.cfg.in
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/lit.site.cfg.in?rev=215209&r1=215208&r2=215209&view=diff
==============================================================================
--- llvm/trunk/test/lit.site.cfg.in (original)
+++ llvm/trunk/test/lit.site.cfg.in Fri Aug  8 10:00:12 2014
@@ -22,6 +22,7 @@ config.host_arch = "@HOST_ARCH@"
 config.llvm_use_intel_jitevents = "@LLVM_USE_INTEL_JITEVENTS@"
 config.llvm_use_sanitizer = "@LLVM_USE_SANITIZER@"
 config.have_zlib = "@HAVE_LIBZ@"
+config.enable_ffi = "@LLVM_ENABLE_FFI@"
 
 # Support substitution of the tools_dir with user parameters. This is
 # used when we can't determine the tool dir at configuration time.