diff --git a/paddle/function/PadOp.cpp b/paddle/function/PadOp.cpp
index 2dfe03dcf61ce36e8fd0ad2b5f248d98b5c3430f..ff13f27e94217b43067391afe53b49cc8f752fbe 100644
--- a/paddle/function/PadOp.cpp
+++ b/paddle/function/PadOp.cpp
@@ -92,9 +92,9 @@ public:
   void calc(const Arguments& inputs,
             const Arguments& outputs,
             const Arguments& inouts) override {
-    CHECK_EQ(1, inputs.size());
-    CHECK_EQ(1, outputs.size());
-    CHECK_EQ(0, inouts.size());
+    CHECK_EQ(1UL, inputs.size());
+    CHECK_EQ(1UL, outputs.size());
+    CHECK_EQ(0UL, inouts.size());
 
     size_t num = inputs[0].dims_[0];
     size_t inC = inputs[0].dims_[1];
@@ -143,9 +143,9 @@ public:
   void calc(const Arguments& inputs,
             const Arguments& outputs,
             const Arguments& inouts) override {
-    CHECK_EQ(1, inputs.size());
-    CHECK_EQ(0, outputs.size());
-    CHECK_EQ(1, inouts.size());
+    CHECK_EQ(1UL, inputs.size());
+    CHECK_EQ(0UL, outputs.size());
+    CHECK_EQ(1UL, inouts.size());
 
     size_t n = inouts[0].dims_[0];
     size_t inC = inouts[0].dims_[1];
diff --git a/paddle/function/PadOpTest.cpp b/paddle/function/PadOpTest.cpp
index c6d573c1d9453709e049a63f60334dfd8f08a53a..be989807d13688c04602884a1c9f9cc663c252e5 100644
--- a/paddle/function/PadOpTest.cpp
+++ b/paddle/function/PadOpTest.cpp
@@ -62,7 +62,7 @@ TEST(PadGrad, real) {
           Dims inDims{numSamples, channels, imgSizeH, imgSizeW};
           Dims outDims{numSamples, channels + 5, imgSizeH + 3, imgSizeW + 5};
           compare.cmpWithArg(
-              {Tensor(nullptr, inDims)}, {}, {Tensor(nullptr, outDims)});
+              {Tensor(nullptr, outDims)}, {}, {Tensor(nullptr, inDims)});
         }
       }
     }
diff --git a/paddle/gserver/tests/test_LayerGrad.cpp b/paddle/gserver/tests/test_LayerGrad.cpp
index 0560bb3ed91d36cd3e3cbdeeaaecd0946a922fbd..8c8e876bd64fb97e11bc04c26ec45358f3f808a1 100644
--- a/paddle/gserver/tests/test_LayerGrad.cpp
+++ b/paddle/gserver/tests/test_LayerGrad.cpp
@@ -850,15 +850,13 @@ TEST(Layer, SequenceLastInstanceLayer) {
   testDegradeLayer(true,
                    "seqlastins",
                    "non-seq");  // hasSubseq seqlastins to non-seq
-  testDegradeLayer(true, "seqlastins", "seq");  // hasSubseq seqlastins to
-  seq
+  testDegradeLayer(true, "seqlastins", "seq");  // hasSubseq seqlastins to seq
 }
 
 TEST(Layer, AverageLayer) {
   testDegradeLayer(false, "average", "non-seq");  // seq average to non-seq
-  testDegradeLayer(true, "average", "non-seq");   // hasSubseq average to
-  non -
-      seq testDegradeLayer(true, "average", "seq");  // hasSubseq average to seq
+  testDegradeLayer(true, "average", "non-seq");  // hasSubseq average to non-seq
+  testDegradeLayer(true, "average", "seq");      // hasSubseq average to seq
 }
 
 TEST(Layer, SequenceConcatLayer) {