"format style"

paddle/operators/mul_op.cc

@@ -66,10 +66,10 @@ class MulOpGrad : public framework::OperatorWithKernel {
     PADDLE_ENFORCE_NOT_NULL(ctx.InputVar("Y"), "Input(Y) should not be null");
     PADDLE_ENFORCE_NOT_NULL(ctx.InputVar(framework::GradVarName("Out")),
                             "Input(Out@GRAD) should not be null");
+    auto x_dims = ctx.Output<Tensor>("X")->dims();
+    auto y_dims = ctx.Output<Tensor>("Y")->dims();
     auto *x_grad = ctx.Output<Tensor>(framework::GradVarName("X"));
     auto *y_grad = ctx.Output<Tensor>(framework::GradVarName("Y"));
-    auto x_dims = ctx.Output<Tensor>(framework::GradVarName("X"))->dims();
-    auto y_dims = ctx.Output<Tensor>(framework::GradVarName("Y"))->dims();
     auto out_dims = ctx.Input<Tensor>(framework::GradVarName("Out"))->dims();
     PADDLE_ENFORCE(x_dims[0] == out_dims[0],
                    "Out@GRAD M X N must equal to X dims 0, M ");

paddle/operators/mul_op.h

@@ -53,7 +53,9 @@ class MulGradKernel : public framework::OpKernel {
     auto* dY = ctx.Output<Tensor>(framework::GradVarName("Y"));
     auto* device_context =
         const_cast<platform::DeviceContext*>(ctx.device_context_);
+    // dX = dOut' * Y. dX: M x K, dOut : M x N, Y : K x N
     math::matmul<Place, T>(*dOut, false, *Y, true, 1, dX, 0, device_context);
+    // dY = X' * dOut. dY: K x N, dOut : M x N, X : M x K
     math::matmul<Place, T>(*X, true, *dOut, false, 1, dY, 0, device_context);
   }
 };