Fix attrbute attribute (#61126)

co63oc · web-flow · commit 3d3d63078ccf · 2024-01-25T19:37:46.000+08:00
diff --git a/paddle/cinn/frontend/op_mappers/science/transform.cc b/paddle/cinn/frontend/op_mappers/science/transform.cc
@@ -68,7 +68,7 @@ void SplitOpMapper(const paddle::cpp::OpDesc& op_desc,
           op_desc, "num_or_sections"));
 
   CHECK(!num_or_sections.empty())
-      << "The Split op cannot found [num_or_sections] attrbute!  ! Please "
+      << "The Split op cannot found [num_or_sections] attribute!  ! Please "
          "check.";
 
   auto axis =
diff --git a/paddle/cinn/frontend/paddle/cpp/op_desc.cc b/paddle/cinn/frontend/paddle/cpp/op_desc.cc
@@ -41,7 +41,7 @@ inline std::string AttrTypeToString(
     EXPAND_SWITCH_CASE(SCALARS)
 #undef EXPAND_SWITCH_CASE
   }
-  return "Invlid AttrType";
+  return "Invalid AttrType";
 }
 
 #define SET_ATTR_IMPL(T, repr__)                                 \
@@ -76,16 +76,16 @@ FindAttr(const OpDesc& desc, const std::string& name) {
   return std::make_pair(it, attr_it);
 }
 
-#define GET_IMPL_ONE(T, repr__)                                              \
-  template <>                                                                \
-  T OpDesc::GetAttr<T>(const std::string& name) const {                      \
-    auto pair = FindAttr(*this, name);                                       \
-    CHECK(pair.second->second == AttrType::repr__)                           \
-        << "The op \"" << Type() << "\"'s attrbute \"" << pair.second->first \
-        << "\"'s type doesn't match the target type! Try get \"" << #repr__  \
-        << "\", but real \"" << AttrTypeToString(pair.second->second)        \
-        << "\". Please check.";                                              \
-    return absl::any_cast<T>(pair.first->second);                            \
+#define GET_IMPL_ONE(T, repr__)                                               \
+  template <>                                                                 \
+  T OpDesc::GetAttr<T>(const std::string& name) const {                       \
+    auto pair = FindAttr(*this, name);                                        \
+    CHECK(pair.second->second == AttrType::repr__)                            \
+        << "The op \"" << Type() << "\"'s attribute \"" << pair.second->first \
+        << "\"'s type doesn't match the target type! Try get \"" << #repr__   \
+        << "\", but real \"" << AttrTypeToString(pair.second->second)         \
+        << "\". Please check.";                                               \
+    return absl::any_cast<T>(pair.first->second);                             \
   }
 
 GET_IMPL_ONE(int32_t, INT);
diff --git a/paddle/cinn/hlir/op/transform.cc b/paddle/cinn/hlir/op/transform.cc
@@ -219,7 +219,7 @@ std::shared_ptr<OpStrategy> StrategyForSplit(
       << "The Spilt Op's output shape list should not empty.";
   CHECK_LT(axis, static_cast<int>(output_shapes[0].size()));
   CHECK(!sections.empty())
-      << "The Split op doesn't find [num_or_sections] attrbute! It it a "
+      << "The Split op doesn't find [num_or_sections] attribute! It it a "
          "mandatory attribute ! Please check.";
 
   framework::CINNCompute split_compute(
@@ -286,8 +286,9 @@ std::vector<std::vector<int>> InferShapeForSplit(
   if (attrs.find("num_or_sections") != attrs.end()) {
     sections = absl::get<std::vector<int>>(attrs.at("num_or_sections"));
   } else {
-    LOG(FATAL) << "The Split op doesn't find [num_or_sections] attrbute! It it "
-                  "a mandatory attribute ! Please check.";
+    LOG(FATAL)
+        << "The Split op doesn't find [num_or_sections] attribute! It it "
+           "a mandatory attribute ! Please check.";
   }
 
   if (inputs_shape.empty()) {
@@ -372,8 +373,9 @@ std::vector<Type> InferDtypeForSplit(const std::vector<Type> &inputs_type,
   if (attrs.find("num_or_sections") != attrs.end()) {
     sections = absl::get<std::vector<int>>(attrs.at("num_or_sections"));
   } else {
-    LOG(FATAL) << "The Split op doesn't find [num_or_sections] attrbute! It it "
-                  "a mandatory attribute ! Please check.";
+    LOG(FATAL)
+        << "The Split op doesn't find [num_or_sections] attribute! It it "
+           "a mandatory attribute ! Please check.";
   }
 
   int output_size = sections.size();
@@ -397,8 +399,9 @@ std::vector<std::vector<std::string>> InferLayoutForSplit(
     sections =
         absl::get<std::vector<int>>(attrs.attr_store.at("num_or_sections"));
   } else {
-    LOG(FATAL) << "The Split op doesn't find [num_or_sections] attrbute! It it "
-                  "a mandatory attribute ! Please check.";
+    LOG(FATAL)
+        << "The Split op doesn't find [num_or_sections] attribute! It it "
+           "a mandatory attribute ! Please check.";
   }
 
   int output_size = sections.size();
@@ -1504,9 +1507,9 @@ std::shared_ptr<OpStrategy> StrategyForSlice(
         absl::get<std::vector<int>>(attrs.attr_store.at("decrease_axis"));
   }
 
-  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attrbute! It "
+  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attribute! It "
                             "it a mandatory attribute, please check.";
-  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attrbute! It it a "
+  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attribute! It it a "
                           "mandatory attribute, please check.";
   CHECK_EQ(starts.size(), ends.size())
       << "The size of [starts] and [ends] must be identical! Please check.";
@@ -1587,9 +1590,9 @@ std::shared_ptr<OpStrategy> StrategyForSliceSymbolic(
         absl::get<std::vector<int>>(attrs.attr_store.at("decrease_axis"));
   }
 
-  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attrbute! It "
+  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attribute! It "
                             "it a mandatory attribute, please check.";
-  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attrbute! It it a "
+  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attribute! It it a "
                           "mandatory attribute, please check.";
   CHECK_EQ(starts.size(), ends.size())
       << "The size of [starts] and [ends] must be identical! Please check.";
@@ -1669,9 +1672,9 @@ std::vector<std::vector<int>> InferShapeForSlice(
       LOG(ERROR) << "Unsupported attr: " << iter.first << std::endl;
     }
   }
-  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attrbute! It "
+  CHECK(!starts.empty()) << "The Slice op doesn't find [starts] attribute! It "
                             "it a mandatory attribute, please check.";
-  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attrbute! It it a "
+  CHECK(!ends.empty()) << "The Slice op doesn't find [ends] attribute! It it a "
                           "mandatory attribute, please check.";
   CHECK_EQ(starts.size(), ends.size())
       << "The size of [starts] and [ends] must be identical! Please check.";
@@ -1824,11 +1827,12 @@ std::shared_ptr<OpStrategy> StrategyForSliceAssign(
     strides = absl::get<std::vector<int>>(attrs.attr_store.at("strides"));
   }
 
-  CHECK(!starts.empty())
-      << "The SliceAssign op doesn't find [starts] attrbute! It it a mandatory "
-         "attribute, please check.";
-  CHECK(!ends.empty()) << "The SliceAssign op doesn't find [ends] attrbute! It "
-                          "it a mandatory attribute, please check.";
+  CHECK(!starts.empty()) << "The SliceAssign op doesn't find [starts] "
+                            "attribute! It it a mandatory "
+                            "attribute, please check.";
+  CHECK(!ends.empty())
+      << "The SliceAssign op doesn't find [ends] attribute! It "
+         "it a mandatory attribute, please check.";
   CHECK_EQ(starts.size(), ends.size())
       << "The size of [starts] and [ends] must be identical! Please check.";
   if (!axes.empty()) {
diff --git a/paddle/fluid/operators/sequence_ops/sequence_expand_op.cc b/paddle/fluid/operators/sequence_ops/sequence_expand_op.cc
@@ -71,7 +71,7 @@ class SequenceExpandOp : public framework::OperatorWithKernel {
               (ref_level >= 0 && ref_level < static_cast<int>(y_lod.size())),
           true,
           platform::errors::InvalidArgument(
-              "Invlid `ref_level`, which should be either equal to -1 "
+              "Invalid `ref_level`, which should be either equal to -1 "
               "or in [0, %d), but received `ref_level` = %u.",
               y_lod.size(),
               ref_level));
