diff --git a/src/framework/operator.h b/src/framework/operator.h
index d9b74e8887944774aefadf5ead0a74b1f7eac79d..26bcf9a784cb71a63dd289fa7198f536e60d45a3 100644
--- a/src/framework/operator.h
+++ b/src/framework/operator.h
@@ -135,7 +135,7 @@ class OpKernelBase {
       const ::paddle_mobile::VariableNameMap &outputs,                         \
       const ::paddle_mobile::framework::AttributeMap &attrs,                   \
       std::shared_ptr<::paddle_mobile::framework::Scope> scope)                \
-      : parent_cls<Dtype, T>(type, inputs, outputs, attrs, scope) {}
+      : parent_cls<Dtype, T>(type, inputs, outputs, attrs, scope){}
 
 class FusionOpMatcher {
  public:
diff --git a/src/operators/batchnorm_op.h b/src/operators/batchnorm_op.h
index 671b2f09f6cbc47570493a8bf1d2c4e23dde9f8b..ea774662abc093c36f75ad693aff579323becb23 100644
--- a/src/operators/batchnorm_op.h
+++ b/src/operators/batchnorm_op.h
@@ -29,7 +29,7 @@ class BatchNormOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   BatchNormOp(const string &type, const VariableNameMap &inputs,
               const VariableNameMap &outputs,
-              const framework::AttributeMap attrs,
+              const framework::AttributeMap &attrs,
               std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/box_coder_op.h b/src/operators/box_coder_op.h
index 2d3cd0d8eaa21df3384a22e0659b10c3eac394a3..001ef20023a4500adec558e6f0bddb16a3c65551 100644
--- a/src/operators/box_coder_op.h
+++ b/src/operators/box_coder_op.h
@@ -32,7 +32,7 @@ class BoxCoderOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   BoxCoderOp(const std::string &type, const VariableNameMap &inputs,
              const VariableNameMap &outputs,
-             const framework::AttributeMap attrs,
+             const framework::AttributeMap &attrs,
              std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/concat_op.h b/src/operators/concat_op.h
index ad2db52a0b65a8474a2534d592b7bbd53924f8cf..fff704e4d858a7c67a0e8331089d8e8d5d4639fb 100644
--- a/src/operators/concat_op.h
+++ b/src/operators/concat_op.h
@@ -27,7 +27,7 @@ template <typename DeviceType, typename T>
 class ConcatOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   ConcatOp(const string &type, const VariableNameMap &inputs,
-           const VariableNameMap &outputs, const framework::AttributeMap attrs,
+           const VariableNameMap &outputs, const framework::AttributeMap &attrs,
            std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/elementwise_add_op.h b/src/operators/elementwise_add_op.h
index 727d569fb3eada6b406c02f127c04022eab4ac2d..62034b14edcbc9ec6ad44af59f6927b8cfa38aa2 100644
--- a/src/operators/elementwise_add_op.h
+++ b/src/operators/elementwise_add_op.h
@@ -29,7 +29,7 @@ class ElementwiseAddOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   ElementwiseAddOp(const string &type, const VariableNameMap &inputs,
                    const VariableNameMap &outputs,
-                   const framework::AttributeMap attrs,
+                   const framework::AttributeMap &attrs,
                    std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/fusion_conv_add.h b/src/operators/fusion_conv_add.h
index 911df63dd4e2a5c00ac364d85dd4916afb72e627..1e6a84b1dc157e0b58273bc5ff379ad079ed2860 100644
--- a/src/operators/fusion_conv_add.h
+++ b/src/operators/fusion_conv_add.h
@@ -50,7 +50,7 @@ class FushionConvAddOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   FushionConvAddOp(const string &type, const VariableNameMap &inputs,
                    const VariableNameMap &outputs,
-                   const framework::AttributeMap attrs,
+                   const framework::AttributeMap &attrs,
                    std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope) {}
diff --git a/src/operators/fusion_fc_op.h b/src/operators/fusion_fc_op.h
index c5419de9ff36898283c3743908b017e01c4c913c..e5c5c04f6938499c1c39ca2e5120ef3084b9c1ad 100644
--- a/src/operators/fusion_fc_op.h
+++ b/src/operators/fusion_fc_op.h
@@ -51,7 +51,7 @@ class FushionFcOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   FushionFcOp(const string &type, const VariableNameMap &inputs,
               const VariableNameMap &outputs,
-              const framework::AttributeMap attrs,
+              const framework::AttributeMap &attrs,
               std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/lrn_op.h b/src/operators/lrn_op.h
index 931c6b4ab069abf9eed496e433a114895f0ace54..c0f7abba0bd095c7408787eda3b819a81fa2227e 100644
--- a/src/operators/lrn_op.h
+++ b/src/operators/lrn_op.h
@@ -28,7 +28,7 @@ template <typename DeviceType, typename T>
 class LrnOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   LrnOp(const string &type, const VariableNameMap &inputs,
-        const VariableNameMap &outputs, const framework::AttributeMap attrs,
+        const VariableNameMap &outputs, const framework::AttributeMap &attrs,
         std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/mul_op.h b/src/operators/mul_op.h
index 85c6c80b925d0be6507bea9a4262a0e6185324a7..5ecf6571ae2725975271d5b0e7212380caa47578 100644
--- a/src/operators/mul_op.h
+++ b/src/operators/mul_op.h
@@ -28,7 +28,7 @@ template <typename DeviceType, typename T>
 class MulOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   MulOp(const std::string &type, const VariableNameMap &inputs,
-        const VariableNameMap &outputs, const framework::AttributeMap attrs,
+        const VariableNameMap &outputs, const framework::AttributeMap &attrs,
         std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/multiclass_nms_op.h b/src/operators/multiclass_nms_op.h
index 78d6ec31204b4a103f59f99cb1aafd5a16ea985a..37f3742524f64b5bfa97f78a55f86b4264489dd5 100644
--- a/src/operators/multiclass_nms_op.h
+++ b/src/operators/multiclass_nms_op.h
@@ -32,7 +32,7 @@ class MultiClassNMSOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   MultiClassNMSOp(const std::string &type, const VariableNameMap &inputs,
                   const VariableNameMap &outputs,
-                  const framework::AttributeMap attrs,
+                  const framework::AttributeMap &attrs,
                   std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/prior_box_op.h b/src/operators/prior_box_op.h
index 55080f3c5a77683acc5ee76fc6ab91545004d010..e3de58b372cc101956d83ff39b02e172c990b254 100644
--- a/src/operators/prior_box_op.h
+++ b/src/operators/prior_box_op.h
@@ -32,7 +32,7 @@ class PriorBoxOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   PriorBoxOp(const std::string &type, const VariableNameMap &inputs,
              const VariableNameMap &outputs,
-             const framework::AttributeMap attrs,
+             const framework::AttributeMap &attrs,
              std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/relu_op.h b/src/operators/relu_op.h
index e52ef5edd2013e30c5004b629c06aa1affe1d20e..f032546c82d740c179385434b5d72082e4bd5a9d 100644
--- a/src/operators/relu_op.h
+++ b/src/operators/relu_op.h
@@ -34,7 +34,7 @@ class ReluOp : public framework::OperatorWithKernel<DeviceType> {
    * @b op 的实例化方法, 需要调用父类的实例化方法, 以及实例化自己的参数结构体
    * */
   ReluOp(const std::string &type, const VariableNameMap &inputs,
-         const VariableNameMap &outputs, const framework::AttributeMap attrs,
+         const VariableNameMap &outputs, const framework::AttributeMap &attrs,
          std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/reshape_op.h b/src/operators/reshape_op.h
index ce106125cb6a5b48c3bc9c03e20c7dbec90c90c0..6f757f2bc57c49954a6fa4fa830e03721cacdf25 100644
--- a/src/operators/reshape_op.h
+++ b/src/operators/reshape_op.h
@@ -31,7 +31,7 @@ template <typename DeviceType, typename T>
 class ReshapeOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   ReshapeOp(const std::string &type, const VariableNameMap &inputs,
-            const VariableNameMap &outputs, const framework::AttributeMap attrs,
+            const VariableNameMap &outputs, const framework::AttributeMap &attrs,
             std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),
diff --git a/src/operators/transpose_op.h b/src/operators/transpose_op.h
index 728c6991c014fcf37b9c0b4f467ccc1c4999883b..f65a725756c858b8e2e304906ed8236b00046fc9 100644
--- a/src/operators/transpose_op.h
+++ b/src/operators/transpose_op.h
@@ -32,7 +32,7 @@ class TransposeOp : public framework::OperatorWithKernel<DeviceType> {
  public:
   TransposeOp(const std::string &type, const VariableNameMap &inputs,
               const VariableNameMap &outputs,
-              const framework::AttributeMap attrs,
+              const framework::AttributeMap &attrs,
               std::shared_ptr<framework::Scope> scope)
       : framework::OperatorWithKernel<DeviceType>(type, inputs, outputs, attrs,
                                                   scope),