Fix some typos (SimplifyNegtive, fill_costant, etc.) (#61919)

co63oc · web-flow · commit 763b9fdac579 · 2024-02-22T10:44:04.000+08:00
diff --git a/paddle/cinn/common/cas_test.cc b/paddle/cinn/common/cas_test.cc
@@ -371,7 +371,7 @@ TEST(CAS, SimplifyCompoundMod) {
 #endif
   }
 }
-TEST(CAS, SimplifyNegtive) {
+TEST(CAS, SimplifyNegative) {
   {  // (-1*x) /2
     Var x = ir::_Var_::Make("x", Int(32));
     auto p0 = ir::FracOp::Make(-x, Expr(2));
diff --git a/paddle/cinn/frontend/pass/fill_constant_folding.cc b/paddle/cinn/frontend/pass/fill_constant_folding.cc
@@ -126,7 +126,7 @@ class FillConstantFoldingPass : public ProgramPass {
       if (!fill_constant_map.count(key)) {
         VLOG(4) << "The fill_constant, whose output is Var ["
                 << instr->outputs[0]->id
-                << "], cannot remove because it is the first fill_costant ! ";
+                << "], cannot remove because it is the first fill_constant ! ";
         // retain the first fill constant op node
         fill_constant_map.emplace(key, &instr->outputs[0]);
         continue;
diff --git a/paddle/cinn/frontend/pass/fill_constant_rewriter.cc b/paddle/cinn/frontend/pass/fill_constant_rewriter.cc
@@ -168,7 +168,7 @@ class FillConstantRewriterPass : public ProgramPass {
     VLOG(3) << "FillConstantRewriterPass Remove " << remove_instr.size()
             << " instruction";
 
-    NetBuilder builder("reshape_rewritter_builder");
+    NetBuilder builder("reshape_rewriter_builder");
     for (auto& var : program->GetInputs()) {
       builder.CreateInput(var);
     }
diff --git a/paddle/cinn/hlir/framework/pir/op_mapper.h b/paddle/cinn/hlir/framework/pir/op_mapper.h
@@ -30,7 +30,7 @@ enum MapperType {
 };
 
 class OpMapper {
-  using OprandIndexsFunction = std::function<std::vector<size_t>()>;
+  using OperandIndexsFunction = std::function<std::vector<size_t>()>;
   using AppendAttrFunction =
       std::function<void(const ::pir::Operation& op,
                          utils::AttributeMap& attrs)>;  // NOLINT
@@ -72,7 +72,7 @@ class OpMapper {
   OpMapper() { RegisterMapRules(); }
   void RegisterMapRules();
 
-  std::unordered_map<std::string, OprandIndexsFunction> operand_funcs_;
+  std::unordered_map<std::string, OperandIndexsFunction> operand_funcs_;
   std::unordered_map<std::string, AppendAttrFunction> attr_funcs_;
 };
 
diff --git a/paddle/fluid/framework/ir/fusion_group/operation.cc b/paddle/fluid/framework/ir/fusion_group/operation.cc
@@ -161,8 +161,8 @@ void OperationMap::InsertBinaryElementwiseOperations() {
                             std::string expr,
                             std::vector<std::string> grad_exprs) {
     int type = 0;
-    int num_oprands = 2;
-    Insert(type, num_oprands, op_type, expr, grad_exprs, {"X", "Y"}, {"Out"});
+    int num_operands = 2;
+    Insert(type, num_operands, op_type, expr, grad_exprs, {"X", "Y"}, {"Out"});
   };
 
   // elementwise_add:
@@ -209,8 +209,8 @@ void OperationMap::InsertMultivariateElementwiseOperations() {
                             std::string expr,
                             std::vector<std::string> grad_exprs) {
     int type = 0;
-    int num_oprands = -1;
-    Insert(type, num_oprands, op_type, expr, grad_exprs, {"X"}, {"Out"});
+    int num_operands = -1;
+    Insert(type, num_operands, op_type, expr, grad_exprs, {"X"}, {"Out"});
   };
 
   // sum:
@@ -226,8 +226,8 @@ void OperationMap::InsertMultivariateElementwiseOperations() {
                                           std::string expr,
                                           std::vector<std::string> grad_exprs) {
     int type = 0;
-    int num_oprands = 0;
-    Insert(type, num_oprands, op_type, expr, grad_exprs, {}, {"Out"});
+    int num_operands = 0;
+    Insert(type, num_operands, op_type, expr, grad_exprs, {}, {"Out"});
   };
   // fill_constant:
   insert_handler_without_input("fill_constant", "${str_value}", {});
diff --git a/paddle/pir/src/core/parser/ir_parser.cc b/paddle/pir/src/core/parser/ir_parser.cc
@@ -116,7 +116,7 @@ Type IrParser::ParseType() {
 // Attribute := BuiltinAttribute | OtherDialectsDefineAttribute
 // BuiltinAttribute := Bool | String | Float | Double | Int32 |
 //                  := | Int64 | Pointer | ArrayAttribute
-// ArrayAttribute   := '[' Atribute(,Attribute)* ']'
+// ArrayAttribute   := '[' Attribute(,Attribute)* ']'
 Attribute IrParser::ParseAttribute() {
   auto parenthesis_token = ConsumeToken();
   if (parenthesis_token.val_ == "true" || parenthesis_token.val_ == "false") {
@@ -204,7 +204,7 @@ void IrParser::ParseBlock(Block& block) {  // NOLINT
   ConsumeAToken("}");
 }
 
-// Operation := ValueList ":=" Opname "(" OprandList ? ")" AttributeMap ":"
+// Operation := ValueList ":=" Opname "(" OperandList ? ")" AttributeMap ":"
 // FunctionType
 // FunctionType := "(" TypeList ")"  "->" TypeList
 Operation* IrParser::ParseOperation() {
@@ -263,7 +263,7 @@ OpInfo IrParser::ParseOpInfo() {
   return ctx->GetRegisteredOpInfo(opname);
 }
 
-// OprandList := ValueList
+// OperandList := ValueList
 // ValueList := ValueId(,ValueId)*
 std::vector<Value> IrParser::ParseOperandList() {
   ConsumeAToken("(");

Original file line number	Diff line number	Diff line change
`@@ -371,7 +371,7 @@ TEST(CAS, SimplifyCompoundMod) {`
`371`	`371`	`#endif`
`372`	`372`	`}`
`373`	`373`	`}`
`374`		`-TEST(CAS, SimplifyNegtive) {`
	`374`	`+TEST(CAS, SimplifyNegative) {`
`375`	`375`	`{ // (-1*x) /2`
`376`	`376`	`Var x = ir::_Var_::Make("x", Int(32));`
`377`	`377`	`auto p0 = ir::FracOp::Make(-x, Expr(2));`
Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ class FillConstantRewriterPass : public ProgramPass {`
`168`	`168`	`VLOG(3) << "FillConstantRewriterPass Remove " << remove_instr.size()`
`169`	`169`	`<< " instruction";`
`170`	`170`
`171`		`- NetBuilder builder("reshape_rewritter_builder");`
	`171`	`+ NetBuilder builder("reshape_rewriter_builder");`
`172`	`172`	`for (auto& var : program->GetInputs()) {`
`173`	`173`	`builder.CreateInput(var);`
`174`	`174`	`}`