diff --git a/paddle/operators/clip_op.cc b/paddle/operators/clip_op.cc
index 8eea843889fbf19689f6baa819f2c228684bfa5b..65bb0ba0165254907c92e5a22b4e03981a2a5569 100644
--- a/paddle/operators/clip_op.cc
+++ b/paddle/operators/clip_op.cc
@@ -26,8 +26,8 @@ class ClipOp : public framework::OperatorWithKernel {
  protected:
   void InferShape(const framework::InferShapeContext &ctx) const override {
     auto x_dims = ctx.Input<Tensor>("X")->dims();
-    auto max = GetAttr<float>("max");
-    auto min = GetAttr<float>("min");
+    auto max = Attr<float>("max");
+    auto min = Attr<float>("min");
     PADDLE_ENFORCE_LT(min, max, "max should be greater than min.");
     ctx.Output<Tensor>("Out")->Resize(x_dims);
   }
diff --git a/paddle/operators/clip_op.cu b/paddle/operators/clip_op.cu
index 51941deecec95a8096e3500c5b9c1e54db00912e..7073fcb023f9a3c4a933e4e165fd5edef5c5c8fa 100644
--- a/paddle/operators/clip_op.cu
+++ b/paddle/operators/clip_op.cu
@@ -34,8 +34,8 @@ template <typename T>
 class ClipGradientOpCUDAKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* d_out = context.Input<Tensor>(framework::GradVarName("Out"));
     auto* d_x = context.Output<Tensor>(framework::GradVarName("X"));
     auto* x = context.Output<Tensor>("X");
diff --git a/paddle/operators/clip_op.h b/paddle/operators/clip_op.h
index b9a2c61f72e988a366b2b078362c3928db7ace10..d596504bd83fda88640eb54577fdeb2c708024eb 100644
--- a/paddle/operators/clip_op.h
+++ b/paddle/operators/clip_op.h
@@ -30,8 +30,8 @@ template <typename Place, typename T>
 class ClipKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* x = context.Input<Tensor>("X");
     auto* out = context.Output<Tensor>("Out");
     out->mutable_data<T>(context.GetPlace());
@@ -46,8 +46,8 @@ template <typename T>
 class ClipGradKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* d_out = context.Input<Tensor>(framework::GradVarName("Out"));
     auto* d_x = context.Output<Tensor>(framework::GradVarName("X"));
     auto* x = context.Output<Tensor>("X");