Fix some typos (optioanl, etc.) (#61836)

co63oc · web-flow · commit 665f97bb6c02 · 2024-02-26T11:17:40.000+08:00
diff --git a/paddle/fluid/framework/new_executor/garbage_collector/event_garbage_collector.cc b/paddle/fluid/framework/new_executor/garbage_collector/event_garbage_collector.cc
@@ -31,8 +31,8 @@ InterpreterCoreEventGarbageCollector::InterpreterCoreEventGarbageCollector(
                            /*allow_spinning*/ true,
                            /*track_task*/ false);
   queue_ = CreateSingleThreadedWorkQueue(options);
-  for (auto& instruc : vec_instruction) {
-    gc_event_.emplace_back(instruc.DeviceContext().GetPlace(),
+  for (auto& instruct : vec_instruction) {
+    gc_event_.emplace_back(instruct.DeviceContext().GetPlace(),
                            platform::GenerateDeviceEventFlag());
   }
 }
@@ -44,8 +44,8 @@ InterpreterCoreEventGarbageCollector::InterpreterCoreEventGarbageCollector(
                            /*allow_spinning*/ true,
                            /*track_task*/ false);
   queue_ = CreateSingleThreadedWorkQueue(options);
-  for (auto& instruc : vec_instruction) {
-    gc_event_.emplace_back(instruc->DeviceContext().GetPlace(),
+  for (auto& instruct : vec_instruction) {
+    gc_event_.emplace_back(instruct->DeviceContext().GetPlace(),
                            platform::GenerateDeviceEventFlag());
   }
 }
diff --git a/paddle/fluid/framework/new_executor/pir_adaptor/pir_adaptor_util.cc b/paddle/fluid/framework/new_executor/pir_adaptor/pir_adaptor_util.cc
@@ -497,7 +497,7 @@ void HandleForSpecialOp(pir::Operation* op,
     }
     PADDLE_ENFORCE(var,
                    paddle::platform::errors::InvalidArgument(
-                       "The variable %s shoud exist", name));
+                       "The variable %s should exist", name));
 
     value_exe_info->Add(value, name);
   } else if (op->isa<pir::CombineOp>()) {
@@ -531,7 +531,7 @@ void HandleForSpecialOp(pir::Operation* op,
                           .AsString();
 
     auto value = op->operand_source(0);
-    // change opreand name to param_name
+    // change operand name to param_name
     auto orig_name = value_exe_info->GetValue2VarName().at(value);
 
     if (param_name == orig_name) {
@@ -547,22 +547,23 @@ void HandleForSpecialOp(pir::Operation* op,
 
     value_exe_info->Rename(param_name, orig_name);
   } else if (op->isa<pir::ShadowOutputOp>()) {
-    VLOG(6) << "Handle for builtin.shadow_ouptut";
+    VLOG(6) << "Handle for builtin.shadow_output";
     auto var_name = op->attributes()
                         .at("output_name")
                         .dyn_cast<pir::StrAttribute>()
                         .AsString();
 
     auto value = op->operand_source(0);
+
     Scope* scope = const_cast<Scope*>(value_exe_info->GetScope());
     if (value.defining_op()->HasAttribute(kAttrIsPersistable) &&
         value.attribute<pir::BoolAttribute>(kAttrIsPersistable).data()) {
-      VLOG(6) << "Handle for builtin.shadow_ouptut persistable value:"
+      VLOG(6) << "Handle for builtin.shadow_output persistable value:"
               << var_name;
       scope = const_cast<Scope*>(value_exe_info->GetScope()->root());
     }
 
-    // change opreand name to param_name
+    // change operand name to param_name
     auto orig_name = value_exe_info->GetValue2VarName().at(value);
 
     if (var_name == orig_name) {
@@ -603,7 +604,7 @@ void HandleForSpecialOp(pir::Operation* op,
     PADDLE_ENFORCE_EQ(value_exe_info->GetValue2VarName().count(in_value),
                       true,
                       phi::errors::PreconditionNotMet(
-                          "input of buildin slice not in name map"));
+                          "input of builtin slice not in name map"));
 
     int index =
         op->attributes().at("index").dyn_cast<pir::Int32Attribute>().data();
@@ -626,7 +627,7 @@ void HandleForSpecialOp(pir::Operation* op,
     PADDLE_ENFORCE_EQ(value_exe_info->GetValue2VarName().count(in_value),
                       true,
                       phi::errors::PreconditionNotMet(
-                          "input of buildin split not in name map"));
+                          "input of builtin split not in name map"));
 
     auto in_var = value_exe_info->GetVarByValue(in_value);
     auto variable_array = in_var->Get<VariableRefArray>();
@@ -817,7 +818,7 @@ void BuildRuntimeContext(pir::Operation* op,
     pir::Value ptr = op->operand_source(index);
 
     if (!IsInvalid(ptr)) {
-      VLOG(8) << "ctx->EmplaceBackInput : an optioanl input " << name;
+      VLOG(8) << "ctx->EmplaceBackInput : an optional input " << name;
       continue;
     }
 
@@ -845,7 +846,7 @@ void BuildRuntimeContext(pir::Operation* op,
     auto legacy_arg_name = op_normalizer.GetLegacyArgName(fluid_op_name, name);
 
     if (!IsInvalid(ptr)) {
-      VLOG(8) << "ctx->EmplaceBackOutput : an optioanl output " << name;
+      VLOG(8) << "ctx->EmplaceBackOutput : an optional output " << name;
       continue;
     }
 
@@ -906,7 +907,7 @@ std::shared_ptr<OperatorBase> BuildOperatorBase(
     auto legacy_attr_name = op_normalizer.GetLegacyArgName(fluid_op_name, name);
 
     if (!IsInvalid(ptr)) {
-      VLOG(8) << "Push back inputs to VariableNameMap : an optioanl input "
+      VLOG(8) << "Push back inputs to VariableNameMap : an optional input "
               << name;
       continue;
     }
@@ -1004,7 +1005,7 @@ std::shared_ptr<OperatorBase> BuildOperatorBase(
         op_normalizer.GetLegacyArgName(fluid_op_name, output_name_list[i]);
 
     if (!IsInvalid(ptr)) {
-      VLOG(8) << "Push back outputs to VariableNameMap : an optioanl output "
+      VLOG(8) << "Push back outputs to VariableNameMap : an optional output "
               << legacy_arg_name;
       continue;
     }
diff --git a/paddle/fluid/framework/new_executor/pir_adaptor/pir_adaptor_util.h b/paddle/fluid/framework/new_executor/pir_adaptor/pir_adaptor_util.h
@@ -190,7 +190,7 @@ void BuildPhiContext(pir::Operation* op,
         InType optional_input(temp);
         ctx->EmplaceBackInput(optional_input);
       }
-      VLOG(8) << "ctx->EmplaceBackInput : an optioanl input " << t;
+      VLOG(8) << "ctx->EmplaceBackInput : an optional input " << t;
       continue;
     }
 
@@ -441,7 +441,7 @@ void BuildPhiContext(pir::Operation* op,
         OutType optional_input(temp);
         ctx->EmplaceBackOutput(optional_input);
       }
-      VLOG(8) << "ctx->EmplaceBackOutput : an optioanl output";
+      VLOG(8) << "ctx->EmplaceBackOutput : an optional output";
       continue;
     }
 
diff --git a/paddle/fluid/pir/transforms/constant_folding_pass.cc b/paddle/fluid/pir/transforms/constant_folding_pass.cc
@@ -74,7 +74,7 @@ class ConstantFoldingPattern : public pir::RewritePattern {
   }
 
   bool Match(pir::Operation* op) const override {
-    VLOG(4) << "constant_folding_pass applys match on [" << op->name()
+    VLOG(4) << "constant_folding_pass applies match on [" << op->name()
             << "] op";
     // 1. Some ops do not need to be processed
     if (op->HasTrait<pir::SideEffectTrait>() ||
@@ -143,7 +143,7 @@ class ConstantFoldingPattern : public pir::RewritePattern {
       }
     }
 
-    // 7. maybe affect performence
+    // 7. maybe affect performance
     if (op->isa<paddle::dialect::FullOp>()) {
       auto next_ops = pir::GetUseOpsForOutput(op, 0);
       for (auto [next_op, _] : next_ops) {
@@ -161,7 +161,7 @@ class ConstantFoldingPattern : public pir::RewritePattern {
 
   void Rewrite(pir::Operation* op,
                pir::PatternRewriter& rewriter) const override {  // NOLINT
-    VLOG(4) << "constant_folding_pass applys rewrite on [" << op->name()
+    VLOG(4) << "constant_folding_pass applies rewrite on [" << op->name()
             << "] op";
     auto output_var_names = RunOp(op, rewriter);
 
@@ -410,7 +410,7 @@ class ConstantFoldingPatternForTrain : public ConstantFoldingPattern {
             context, suffix, place, scope, exe_config, deleted_vars) {}
 
   bool Match(pir::Operation* op) const override {
-    VLOG(4) << "constant_folding_pass applys match on [" << op->name()
+    VLOG(4) << "constant_folding_pass applies match on [" << op->name()
             << "] op";
     if (!ConstantFoldingPattern::Match(op)) {
       return false;
@@ -427,7 +427,7 @@ class ConstantFoldingPatternForTrain : public ConstantFoldingPattern {
 
   void Rewrite(pir::Operation* op,
                pir::PatternRewriter& rewriter) const override {  // NOLINT
-    VLOG(4) << "constant_folding_pass for train applys rewrite on ["
+    VLOG(4) << "constant_folding_pass for train applies rewrite on ["
             << op->name() << "] op";
 
     auto output_var_names = RunOp(op, rewriter);
@@ -454,7 +454,7 @@ class ConstantFoldingPatternForTrain : public ConstantFoldingPattern {
       rewriter.ReplaceAllUsesWith(op->result(i), constant_op->result(0));
     }
     rewriter.EraseOp(op);
-    VLOG(4) << "constant_folding_pass for traun applied rewrite on ["
+    VLOG(4) << "constant_folding_pass for train applied rewrite on ["
             << op->name() << "] op";
   }
 };
diff --git a/paddle/phi/api/yaml/generator/dist_api_gen.py b/paddle/phi/api/yaml/generator/dist_api_gen.py
@@ -36,7 +36,7 @@
   // Kernel Dispatch Body{}
 }}
 """
-DIPATCH_END_GUARD_TEMPLATE = """
+DISPATCH_END_GUARD_TEMPLATE = """
 PADDLE_THROW(phi::errors::Unimplemented(
           "The kernel of ({}) for input tensors is unimplemented, please check the type of input tensors."));
 """
@@ -1899,7 +1899,7 @@ def gene_base_api_code(self, inplace_flag=False):
                 self.get_define_args(inplace_flag),
                 self.gene_kernel_select(),
                 kernel_dispatch_code
-                + DIPATCH_END_GUARD_TEMPLATE.format(self.api),
+                + DISPATCH_END_GUARD_TEMPLATE.format(self.api),
             )
         else:
             dist_branch_code = ""
@@ -1947,7 +1947,7 @@ def generate_api(
         if is_fused_ops_yaml is True
         else "paddle/phi/api/include/api.h"
     )
-    # not all fused ops supoort dygraph
+    # not all fused ops support dygraph
     if is_fused_ops_yaml is True:
         new_apis = [
             api

Original file line number	Diff line number	Diff line change
`@@ -31,8 +31,8 @@ InterpreterCoreEventGarbageCollector::InterpreterCoreEventGarbageCollector(`
`31`	`31`	`/allow_spinning/ true,`
`32`	`32`	`/track_task/ false);`
`33`	`33`	`queue_ = CreateSingleThreadedWorkQueue(options);`
`34`		`- for (auto& instruc : vec_instruction) {`
`35`		`- gc_event_.emplace_back(instruc.DeviceContext().GetPlace(),`
	`34`	`+ for (auto& instruct : vec_instruction) {`
	`35`	`+ gc_event_.emplace_back(instruct.DeviceContext().GetPlace(),`
`36`	`36`	`platform::GenerateDeviceEventFlag());`
`37`	`37`	`}`
`38`	`38`	`}`
`@@ -44,8 +44,8 @@ InterpreterCoreEventGarbageCollector::InterpreterCoreEventGarbageCollector(`
`44`	`44`	`/allow_spinning/ true,`
`45`	`45`	`/track_task/ false);`
`46`	`46`	`queue_ = CreateSingleThreadedWorkQueue(options);`
`47`		`- for (auto& instruc : vec_instruction) {`
`48`		`- gc_event_.emplace_back(instruc->DeviceContext().GetPlace(),`
	`47`	`+ for (auto& instruct : vec_instruction) {`
	`48`	`+ gc_event_.emplace_back(instruct->DeviceContext().GetPlace(),`
`49`	`49`	`platform::GenerateDeviceEventFlag());`
`50`	`50`	`}`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ void BuildPhiContext(pir::Operation* op,`
`190`	`190`	`InType optional_input(temp);`
`191`	`191`	`ctx->EmplaceBackInput(optional_input);`
`192`	`192`	`}`
`193`		`- VLOG(8) << "ctx->EmplaceBackInput : an optioanl input " << t;`
	`193`	`+ VLOG(8) << "ctx->EmplaceBackInput : an optional input " << t;`
`194`	`194`	`continue;`
`195`	`195`	`}`
`196`	`196`
`@@ -441,7 +441,7 @@ void BuildPhiContext(pir::Operation* op,`
`441`	`441`	`OutType optional_input(temp);`
`442`	`442`	`ctx->EmplaceBackOutput(optional_input);`
`443`	`443`	`}`
`444`		`- VLOG(8) << "ctx->EmplaceBackOutput : an optioanl output";`
	`444`	`+ VLOG(8) << "ctx->EmplaceBackOutput : an optional output";`
`445`	`445`	`continue;`
`446`	`446`	`}`
`447`	`447`