From 6f932482f435f7f80c176afbd9f429c09bce381f Mon Sep 17 00:00:00 2001
From: Tomasz Patejko <tomasz.patejko@intel.com>
Date: Sat, 19 May 2018 16:48:29 +0200
Subject: [PATCH] MKL elementwise_add: BLAS version compiles with integral
 types

---
 paddle/fluid/operators/elementwise_add_op.cc |  8 ++--
 paddle/fluid/operators/elementwise_add_op.h  | 39 +++++++++++++++-----
 2 files changed, 34 insertions(+), 13 deletions(-)

diff --git a/paddle/fluid/operators/elementwise_add_op.cc b/paddle/fluid/operators/elementwise_add_op.cc
index d51a845b4..d2c205371 100644
--- a/paddle/fluid/operators/elementwise_add_op.cc
+++ b/paddle/fluid/operators/elementwise_add_op.cc
@@ -18,10 +18,10 @@ namespace ops = paddle::operators;
 REGISTER_ELEMWISE_OP(elementwise_add, "Add", "Out = X + Y");
 REGISTER_OP_CPU_KERNEL(
     elementwise_add,
-    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, float>);
-//    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, double>);
-//    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, int>,
-//    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, int64_t>);
+    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, float>,
+    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, double>,
+    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, int>,
+    ops::ElementwiseAddKernel<paddle::platform::CPUDeviceContext, int64_t>);
 REGISTER_OP_CPU_KERNEL(
     elementwise_add_grad,
     ops::ElementwiseAddGradKernel<paddle::platform::CPUDeviceContext, float>,
diff --git a/paddle/fluid/operators/elementwise_add_op.h b/paddle/fluid/operators/elementwise_add_op.h
index 316fd7568..1f8735b7b 100644
--- a/paddle/fluid/operators/elementwise_add_op.h
+++ b/paddle/fluid/operators/elementwise_add_op.h
@@ -26,6 +26,34 @@ struct AddFunctor {
   inline HOSTDEVICE T operator()(T a, T b) const { return a + b; }
 };
 
+template <typename DeviceContext, typename T>
+void default_elementwise_add(const framework::ExecutionContext& ctx,
+                             const framework::Tensor* x,
+                             const framework::Tensor* y, framework::Tensor* z) {
+  int axis = ctx.Attr<int>("axis");
+  ElementwiseComputeEx<AddFunctor<T>, DeviceContext, T>(ctx, x, y, axis,
+                                                        AddFunctor<T>(), z);
+}
+
+template <typename DeviceContext, typename T>
+typename std::enable_if<std::is_floating_point<T>::value>::type elementwise_add(
+    const framework::ExecutionContext& ctx, const framework::Tensor* x,
+    const framework::Tensor* y, framework::Tensor* z) {
+  auto eigen_x = framework::EigenVector<T>::Flatten(*x);
+  auto eigen_y = framework::EigenVector<T>::Flatten(*y);
+  auto eigen_z = framework::EigenVector<T>::Flatten(*z);
+
+  auto blas = math::GetBlas<DeviceContext, T>(ctx);
+  blas.VADD(x->numel(), eigen_x.data(), eigen_y.data(), eigen_z.data());
+}
+
+template <typename DeviceContext, typename T>
+typename std::enable_if<std::is_integral<T>::value>::type elementwise_add(
+    const framework::ExecutionContext& ctx, const framework::Tensor* x,
+    const framework::Tensor* y, framework::Tensor* z) {
+  default_elementwise_add<DeviceContext, T>(ctx, x, y, z);
+}
+
 template <typename DeviceContext, typename T>
 class ElementwiseAddKernel : public framework::OpKernel<T> {
  public:
@@ -36,19 +64,12 @@ class ElementwiseAddKernel : public framework::OpKernel<T> {
     const auto y = ctx.Input<Tensor>("Y");
     auto z = ctx.Output<Tensor>("Out");
     z->mutable_data<T>(ctx.GetPlace());
-    int axis = ctx.Attr<int>("axis");
 
     auto dims_equal = x->dims() == y->dims();
     if (platform::is_cpu_place(ctx.GetPlace()) && dims_equal) {
-      auto eigen_x = framework::EigenVector<T>::Flatten(*x);
-      auto eigen_y = framework::EigenVector<T>::Flatten(*y);
-      auto eigen_z = framework::EigenVector<T>::Flatten(*z);
-
-      auto blas = math::GetBlas<DeviceContext, T>(ctx);
-      blas.VADD(x->numel(), eigen_x.data(), eigen_y.data(), eigen_z.data());
+      elementwise_add<DeviceContext, T>(ctx, x, y, z);
     } else {
-      ElementwiseComputeEx<AddFunctor<T>, DeviceContext, T>(ctx, x, y, axis,
-                                                            AddFunctor<T>(), z);
+      default_elementwise_add<DeviceContext, T>(ctx, x, y, z);
     }
   }
 };
-- 
GitLab