From fc385777e43344c663754bacacb72557db6b13b3 Mon Sep 17 00:00:00 2001
From: wangchaochaohu <wangchao66@baidu.com>
Date: Fri, 8 Nov 2019 17:44:25 +0800
Subject: [PATCH] fix the compile cost long time  test=develop (#21064)

---
 paddle/fluid/operators/expand_as_op.h | 39 ++++++++++++---------------
 1 file changed, 17 insertions(+), 22 deletions(-)
 mode change 100755 => 100644 paddle/fluid/operators/expand_as_op.h

diff --git a/paddle/fluid/operators/expand_as_op.h b/paddle/fluid/operators/expand_as_op.h
old mode 100755
new mode 100644
index 249f4c35a7f..b189aa6f122
--- a/paddle/fluid/operators/expand_as_op.h
+++ b/paddle/fluid/operators/expand_as_op.h
@@ -31,9 +31,7 @@ limitations under the License. */
     break;                             \
   }
 #define REP_EXPAND_AS_TEMPLATE(n) BOOST_PP_REPEAT(n, EXPAND_AS_TEMPLATE, ~)
-#define COND(n)                                               \
-  BOOST_PP_GREATER_EQUAL(BOOST_PP_DIV(n, MAX_RANK_SUPPORTED), \
-                         BOOST_PP_MOD(n, MAX_RANK_SUPPORTED))
+#define COND(n) BOOST_PP_GREATER_EQUAL(n, BOOST_PP_MOD(n, MAX_RANK_SUPPORTED))
 #define EXPAND_AS_GRAD_CASE(n)                                       \
   case n: {                                                          \
     ExpandAsBackward<n>(context, reshape_dims_vec, reduce_dims_vec); \
@@ -116,23 +114,20 @@ class ExpandAsGradKernel : public framework::OpKernel<T> {
     std::vector<int> reshape_dims_vec;
     std::vector<int> reduce_dims_vec;
     for (size_t i = 0; i < bcast_dims.size(); ++i) {
-      if (bcast_dims[i] == 1) {
-        reshape_dims_vec.push_back(x_dims[i]);
-      } else {
-        if (x_dims[i] == 1) {
-          reduce_dims_vec.push_back(reshape_dims_vec.size());
-          reshape_dims_vec.push_back(bcast_dims[i]);
-        } else {
-          reduce_dims_vec.push_back(reshape_dims_vec.size());
-          reshape_dims_vec.push_back(bcast_dims[i]);
-          reshape_dims_vec.push_back(x_dims[i]);
-        }
+      reduce_dims_vec.push_back(reshape_dims_vec.size());
+      reshape_dims_vec.push_back(bcast_dims[i]);
+      reshape_dims_vec.push_back(x_dims[i]);
+    }
+    int dims = reduce_dims_vec.size();
+    bool just_copy = true;
+    for (size_t i = 0; i < bcast_dims.size(); i++) {
+      if (bcast_dims[i] != 1) {
+        just_copy = false;
+        break;
       }
     }
-    int dims = reshape_dims_vec.size() * MAX_RANK_SUPPORTED +
-               reduce_dims_vec.size() - MAX_RANK_SUPPORTED - 1;
     // no need reduce, just copy
-    if (reduce_dims_vec.size() == 0) {
+    if (just_copy) {
       auto* in0 = context.Input<Tensor>(framework::GradVarName("Out"));
       auto* out0 = context.Output<Tensor>(framework::GradVarName("X"));
       out0->mutable_data<T>(context.GetPlace());
@@ -140,7 +135,7 @@ class ExpandAsGradKernel : public framework::OpKernel<T> {
                             out0);
     } else {
       switch (dims) {
-        REP_EXPAND_AS_GRAD_TEMPLATE(72)
+        REP_EXPAND_AS_GRAD_TEMPLATE(MAX_RANK_SUPPORTED)
         default:
           PADDLE_THROW("Only support tensor with rank being between 1 and 6.");
       }
@@ -152,8 +147,8 @@ class ExpandAsGradKernel : public framework::OpKernel<T> {
   void ExpandAsBackward(const framework::ExecutionContext& context,
                         const std::vector<int>& reshape_dims_vec,
                         const std::vector<int>& reduce_dims_vec) const {
-    size_t reshape_size = Dims / MAX_RANK_SUPPORTED + 1;
-    size_t reduce_size = Dims % MAX_RANK_SUPPORTED + 1;
+    size_t reshape_size = reshape_dims_vec.size();
+    size_t reduce_size = reduce_dims_vec.size();
     PADDLE_ENFORCE_EQ(reshape_size, reshape_dims_vec.size(),
                       "Inconsistent size between template Dims and "
                       "reshape dimensions.");
@@ -164,11 +159,11 @@ class ExpandAsGradKernel : public framework::OpKernel<T> {
     auto* out0 = context.Output<Tensor>(framework::GradVarName("X"));
     out0->mutable_data<T>(context.GetPlace());
     auto x_grad = EigenVector<T>::Flatten(*out0);
-    Eigen::DSizes<int, Dims / MAX_RANK_SUPPORTED + 1> reshape_dims;
+    Eigen::DSizes<int, Dims * 2> reshape_dims;
     for (size_t i = 0; i < reshape_size; ++i) {
       reshape_dims[i] = reshape_dims_vec[i];
     }
-    Eigen::DSizes<int, Dims % MAX_RANK_SUPPORTED + 1> reduce_dims;
+    Eigen::DSizes<int, Dims> reduce_dims;
     for (size_t i = 0; i < reduce_size; ++i) {
       reduce_dims[i] = reduce_dims_vec[i];
     }
-- 
GitLab