Polish code

paddle/framework/backward.cc

@@ -203,7 +203,6 @@ static std::unique_ptr<OperatorBase> BackwardRecursive(
     }
   } else {
     std::unique_ptr<OperatorBase> grad_op(CreateGradOp(forwardOp));
-    PADDLE_ENFORCE(grad_op != nullptr);
 
     ForEachVarName(grad_op->Inputs(), [&no_grad_names, &net, &grad_op](
                                           const std::string& grad_input) {

paddle/framework/backward_test.cc

@@ -171,17 +171,6 @@ REGISTER_OP_WITHOUT_GRADIENT(fc, f::FcOp, f::FcOpMaker);
 REGISTER_OP(many_output_op, f::NOP, f::ManyOutputOpMaker, many_output_op_grad,
             f::NOP);
 
-// TEST(Backward, simple_op_grad) {
-//  auto fwd = f::OpRegistry::CreateOp(
-//      "rowwise_add", {{"X", {"x"}}, {"b", {"b"}}}, {{"Out", {"out"}}}, {});
-//  ASSERT_NE(fwd, nullptr);
-//  auto gop = f::OpRegistry::CreateGradOp(*fwd);
-//  ASSERT_EQ(1UL, gop->Inputs().size());
-//  ASSERT_EQ("rowwise_add_grad", gop->Type());
-//  ASSERT_EQ(f::GradVarName("x"), gop->Output(f::GradVarName("X")));
-//  ASSERT_EQ(f::GradVarName("b"), gop->Output(f::GradVarName("b")));
-//}
-
 TEST(Backward, simple_op_not_need_grad) {
   auto fwd = f::OpRegistry::CreateOp(
       "rowwise_add", {{"X", {"x"}}, {"b", {"b"}}}, {{"Out", {"out"}}}, {});
@@ -390,7 +379,6 @@ TEST(Backward, linear_net_intermediate_variable_has_no_grad) {
                 + 1UL /* external output number*/
                 + 1UL /* number of gradient of external output*/
                 + 2U /* internal variable number*/);
-
   EXPECT_EQ(grad_fc.Outputs(all).size(),
             2UL       /* input number of mul*/
                 + 2UL /* input number of rowwise_add

paddle/framework/op_info.h

@@ -23,8 +23,6 @@
 #include "paddle/framework/type_defs.h"
 #include "paddle/platform/macros.h"
 
-#include "glog/logging.h"
-
 namespace paddle {
 namespace framework {
 

paddle/framework/op_registry.cc

@@ -55,7 +55,6 @@ std::unique_ptr<OperatorBase> OpRegistry::CreateOp(const OpDesc& op_desc) {
 }
 
 std::unique_ptr<OperatorBase> OpRegistry::CreateOp(OpDescBind* op_desc) {
-  op_desc->Sync();
   return CreateOp(op_desc->Type(), op_desc->Inputs(), op_desc->Outputs(),
                   op_desc->GetAttrMap());
 }

paddle/operators/mean_op.cc

@@ -71,7 +71,6 @@ class MeanGradMaker : public framework::SingleGradOpDescMaker {
 }  // namespace paddle
 
 namespace ops = paddle::operators;
-
 REGISTER_OPERATOR(mean, ops::MeanOp, ops::MeanOpMaker, ops::MeanGradMaker);
 REGISTER_OPERATOR(mean_grad, ops::MeanGradOp);
 REGISTER_OP_CPU_KERNEL(mean,

paddle/operators/softmax_with_cross_entropy_op.cc

@@ -16,11 +16,6 @@
 #include <paddle/function/TensorType.h>
 #include <iostream>
 
-#define DBG_LINE()                                   \
-  do {                                               \
-    std::cerr << "Run at " << __LINE__ << std::endl; \
-  } while (false)
-
 namespace paddle {
 namespace operators {