Fix caculation calculation (#61264)

co63oc · web-flow · commit deb52f5b7af7 · 2024-01-30T11:03:06.000+08:00
diff --git a/paddle/cinn/common/axis.h b/paddle/cinn/common/axis.h
@@ -32,7 +32,7 @@ struct Expr;
 namespace cinn {
 namespace common {
 
-//! Get the predifined axis name.
+//! Get the predefined axis name.
 std::string axis_name(int level);
 
 //! Generate `naxis` axis using the global names (i,j,k...).
diff --git a/paddle/cinn/common/bfloat16.h b/paddle/cinn/common/bfloat16.h
@@ -158,7 +158,7 @@ struct CINN_ALIGN(2) bfloat16 {
     return *this;
   }
 
-  // Conversion opertors
+  // Conversion operators
   __host__ __device__ inline operator float() const {
 #ifdef CINN_CUDA_BF16
     return __bfloat162float(*reinterpret_cast<const __nv_bfloat16*>(&x));
diff --git a/paddle/cinn/common/cas.cc b/paddle/cinn/common/cas.cc
@@ -73,7 +73,7 @@ int gcd(int a, int b) {
 }
 
 //////// All the following symbolic computation methods are implemented
-/// referencing to the book <Computer Algegra and
+/// referencing to the book <Computer Algebra and
 /// Symbolic Computation - Joel S. Cohen>
 
 template <typename T>
@@ -1348,7 +1348,7 @@ Expr CasSimplifyMutator::SimplifyMinAndMax(Expr u) {
           return const_operand;
         }
       }
-      // not unfold var for var may be eliminated in the caculation
+      // not unfold var for var may be eliminated in the calculation
       if (GetExprBound(&lower_bound, &upper_bound, non_const_operand, false)) {
         // if non_const_operand's lower_bound is larger than const_operand, then
         // non_const_operand must be larger than const_operand
@@ -1448,10 +1448,10 @@ Expr CasSimplifyMutator::SimplifyCmp(Expr u) {
 }
 
 /**
- * deal with index's div-mod add simplification, tempory solution, not cover all
- * situations. case 1: (m / n) * n + m % n = m (m, n's type is int) case 2: (m /
- * n1) * n3 + (n2 * m) % n3 = n2 * m if n3 = n1 * n2 (m, n1, n2, n3's type is
- * int)
+ * deal with index's div-mod add simplification, temporary solution, not cover
+ * all situations. case 1: (m / n) * n + m % n = m (m, n's type is int) case 2:
+ * (m / n1) * n3 + (n2 * m) % n3 = n2 * m if n3 = n1 * n2 (m, n1, n2, n3's type
+ * is int)
  */
 Expr CasSimplifyMutator::SimplifySpecificSum(Expr tmp) {
   auto sum = tmp.As<Sum>();
@@ -1546,11 +1546,11 @@ Expr CasSimplifyMutator::operator()(Expr u) {
 
   if (u.As<Sum>()) {
     auto tmp = detail::SumOrProductGetSingleElementsRec(SimplifySum(u));
-    // deal with index's div-mod add simplification, tempory solution, not cover
-    // all situations. case 1: (m / n) * n + m % n = m (m, n's type is int) case
-    // 2: (m / n1) * n3 + (n2 * m) % n3 = n2 * m if n3 = n1 * n2 (m, n1, n2,
-    // n3's type is int) case 3: m / n2 + (n1 * m) % n3 = n1 * m if n3 = n1 * n2
-    // (m, n1, n2, n3's type is int)
+    // deal with index's div-mod add simplification, temporary solution, not
+    // cover all situations. case 1: (m / n) * n + m % n = m (m, n's type is
+    // int) case 2: (m / n1) * n3 + (n2 * m) % n3 = n2 * m if n3 = n1 * n2 (m,
+    // n1, n2, n3's type is int) case 3: m / n2 + (n1 * m) % n3 = n1 * m if n3 =
+    // n1 * n2 (m, n1, n2, n3's type is int)
     return SimplifySpecificSum(tmp);
   }
 
diff --git a/paddle/cinn/common/equation_graph_topo_walker.h b/paddle/cinn/common/equation_graph_topo_walker.h
@@ -123,7 +123,7 @@ class EquationGraphTopoWalker final {
     std::unordered_set<VT> queued_variables{};
     std::queue<FT> functions_queue{};
     std::unordered_set<FT> queued_functions{};
-    const auto& TryEnqueueVaraible = [&](VT variable) {
+    const auto& TryEnqueueVariable = [&](VT variable) {
       if (queued_variables.count(variable) == 0) {
         variables_queue.push(variable);
         queued_variables.insert(variable);
@@ -136,14 +136,14 @@ class EquationGraphTopoWalker final {
       }
     };
     for (VarIterT iter = begin; iter != end; ++iter) {
-      TryEnqueueVaraible(*iter);
+      TryEnqueueVariable(*iter);
     }
     while (!functions_queue.empty() || !variables_queue.empty()) {
       if (!functions_queue.empty()) {
         FT function = functions_queue.front();
         functions_queue.pop();
         FunctionVisitor(function);
-        VisitOutputVariables(function, TryEnqueueVaraible);
+        VisitOutputVariables(function, TryEnqueueVariable);
       }
       if (!variables_queue.empty()) {
         VT variable = variables_queue.front();
diff --git a/paddle/cinn/common/float16.h b/paddle/cinn/common/float16.h
@@ -195,7 +195,7 @@ struct CINN_ALIGN(2) float16 {
     return *this;
   }
 
-// Conversion opertors
+// Conversion operators
 #ifdef CINN_CUDA_FP16
   __host__ __device__ inline half to_half() const {
 #if CUDA_VERSION >= 9000
diff --git a/paddle/cinn/common/graph_utils.cc b/paddle/cinn/common/graph_utils.cc
@@ -76,7 +76,7 @@ Graph::topological_order() const {
   std::vector<GraphEdge *> edge_order;
   std::deque<GraphNode *> queue;
 
-  // collect indegreee.
+  // collect indegree.
   std::map<std::string, int> indegree;
   for (auto *n : nodes()) {
     indegree[n->id()] = n->inlinks().size();
diff --git a/paddle/cinn/common/scc_walker.h b/paddle/cinn/common/scc_walker.h
@@ -28,7 +28,7 @@
 namespace cinn {
 namespace common {
 
-// strong connnected components visitor
+// strong connected components visitor
 template <typename NodeType>
 class SccWalker final {
  public:
@@ -75,7 +75,7 @@ class SccWalker final {
         continue;
       }
       std::vector<NodeType> scc;
-      // Use node2root immutablely inside dfs visitor.
+      // Use node2root immutably inside dfs visitor.
       DfsVisitor<NodeType> visitor(VisitPrevNode);
       visitor(root, [&](NodeType node) { scc.push_back(node); });
       SccHandler(scc);
diff --git a/paddle/cinn/frontend/decomposer/broadcast.cc b/paddle/cinn/frontend/decomposer/broadcast.cc
@@ -72,7 +72,7 @@ void elementwise_add(const Instruction& instr,
     Variable bcast_x = x;
     Variable bcast_y = y;
 
-    // e.g., x.shape = [4, 1, 3], y.shape = [2, 3], aixs = 1 out.shape = [4, 2,
+    // e.g., x.shape = [4, 1, 3], y.shape = [2, 3], axis = 1 out.shape = [4, 2,
     // 3] bcast_axes_x = [0, 1, 2], bcast_axes_y = [1, 2]
     if (x->shape != output->shape) {
       std::vector<int> bcast_axes_x(x->shape.size());
diff --git a/paddle/cinn/frontend/op_mappers/paddle/argsort.cc b/paddle/cinn/frontend/op_mappers/paddle/argsort.cc
@@ -46,8 +46,8 @@ void ArgsortOpMapper(const paddle::cpp::OpDesc& op_desc,
   ctx.AddVarModelToProgram(indices_name, idx->id);
 
   // TODO(lanxianghit): return the sorted tensor here. Now out[1] is a temporary
-  // tensor. this is because output 'Out' is never uesd in Paddle API, but CINN
-  // need to return 2 output vars to meet the op defination, this should be
+  // tensor. this is because output 'Out' is never used in Paddle API, but CINN
+  // need to return 2 output vars to meet the op definition, this should be
   // resolved after sort op restructured.
   ctx.AddVar(out_name, out[1]);
   ctx.AddVarModelToProgram(out_name, out[1]->id);
diff --git a/paddle/cinn/frontend/op_mappers/paddle/batchnorm.cc b/paddle/cinn/frontend/op_mappers/paddle/batchnorm.cc
@@ -97,7 +97,7 @@ void BatchNormOpMapper(const paddle::cpp::OpDesc& op_desc,
     auto save_variance = ctx.Builder()->Identity(variance);
     add_output("SavedVariance", save_variance);
     // Just for skip error of "Variable(batch_norm2d_0.w_2@InplaceOut) not
-    // applied in cinn" when run batchnorm in paddle, remove after inpace
+    // applied in cinn" when run batchnorm in paddle, remove after inplace
     // mechanism perfect. The value should shared memory with mean and variance.
     auto mean_out = ctx.Builder()->Identity(mean);
     add_output("MeanOut", mean_out, true);
diff --git a/paddle/cinn/frontend/op_mappers/paddle/tile.cc b/paddle/cinn/frontend/op_mappers/paddle/tile.cc
@@ -56,7 +56,7 @@ void TileOpMapper(const paddle::cpp::OpDesc& op_desc,
   // output's shape
   std::vector<int> output_shape = vec_x_dims;
 
-  // calucate output's shape
+  // calculate output's shape
   for (size_t i = 0; i < repeat_times.size(); ++i) {
     output_shape[i] *= repeat_times[i];
   }
@@ -71,15 +71,15 @@ void TileOpMapper(const paddle::cpp::OpDesc& op_desc,
 
   // make a copy of vec_x_dims
   std::vector<int> vec_x_dims_copy = vec_x_dims;
-  // recontruct vec_x_dims_copy by inserting 1 before every element
+  // reconstruct vec_x_dims_copy by inserting 1 before every element
   for (size_t i = 0; i < vec_x_dims_copy.size(); ++i) {
     vec_x_dims_copy.insert(vec_x_dims_copy.begin() + i, 1);
     i++;
   }
 
   x = ctx.Builder()->Reshape(x, vec_x_dims_copy);
 
-  // recontruct vec_x_dims_copy for BroadaCast
+  // reconstruct vec_x_dims_copy for BroadCast
   for (size_t i = 0; i < vec_x_dims_copy.size(); ++i) {
     if (i % 2 == 0) {
       vec_x_dims_copy[i] = output_shape[i / 2] / vec_x_dims_copy[i + 1];
diff --git a/paddle/cinn/frontend/op_mappers/science/transform.cc b/paddle/cinn/frontend/op_mappers/science/transform.cc
@@ -254,7 +254,7 @@ void ReduceOpMapper(const paddle::cpp::OpDesc& op_desc,
 
   auto x = ctx.GetVar(x_name);
 
-  VLOG(4) << "Reudce " << reduce_type << " x:" << x_name << " from shape ("
+  VLOG(4) << "Reduce " << reduce_type << " x:" << x_name << " from shape ("
           << cinn::utils::Join(x->shape, ",") << "), with axis ["
           << cinn::utils::Join(axis, ",") << "], keepdim " << keepdim;
 
diff --git a/paddle/cinn/frontend/paddle/compatible_pb.cc b/paddle/cinn/frontend/paddle/compatible_pb.cc
@@ -25,7 +25,7 @@
 namespace cinn::frontend::paddle {
 namespace framework_proto = ::cinn::frontend::paddle::proto;
 
-/// For VarDesc transfrom
+/// For VarDesc transform
 #define TRANS_VAR_ANY_WITH_CPP_IMPL(T)                             \
   template <>                                                      \
   void TransformVarDescCppToAny<T>(const cpp::VarDesc &cpp_desc,   \
diff --git a/paddle/cinn/frontend/paddle/cpp/block_desc.h b/paddle/cinn/frontend/paddle/cpp/block_desc.h
@@ -24,7 +24,7 @@ namespace cinn::frontend::paddle::cpp {
 
 /*
  * The cpp::BlockDesc is the internal representation for Op. All the internal
- * imprementation should use it, not the pb::BlockDesc.
+ * implementation should use it, not the pb::BlockDesc.
  */
 class BlockDesc : public BlockDescAPI {
  public:
diff --git a/paddle/cinn/frontend/paddle/cpp/desc_api.h b/paddle/cinn/frontend/paddle/cpp/desc_api.h
@@ -98,7 +98,7 @@ class VarDescAPI {
 };
 
 /*
- * NOTE Some interfaces are weried, we remain them unchanged to keep compatible
+ * NOTE Some interfaces are wired, we remain them unchanged to keep compatible
  * with framework::OpDesc in Fluid framework.
  */
 class OpDescAPI {
diff --git a/paddle/cinn/frontend/paddle/cpp/program_desc.h b/paddle/cinn/frontend/paddle/cpp/program_desc.h
@@ -24,7 +24,7 @@ namespace cinn::frontend::paddle::cpp {
 
 /*
  * The cpp::ProgramDesc is the internal representation for Op. All the internal
- * imprementation should use it, not the pb::ProgramDesc.
+ * implementation should use it, not the pb::ProgramDesc.
  */
 class ProgramDesc : public ProgramDescAPI {
  public:
@@ -43,7 +43,7 @@ class ProgramDesc : public ProgramDescAPI {
   template <typename T>
   T* AddBlock();
 
-  // Just return default versoin
+  // Just return default version
   // TODO(sangoly): refine this
   bool HasVersion() const override { return true; }
 
diff --git a/paddle/cinn/frontend/paddle/cpp/var_desc.h b/paddle/cinn/frontend/paddle/cpp/var_desc.h
@@ -19,7 +19,7 @@ namespace cinn::frontend::paddle::cpp {
 
 /*
  * The cpp::VarDesc is the internal representation for Op. All the internal
- * imprementation should use it, not the pb::VarDesc.
+ * implementation should use it, not the pb::VarDesc.
  */
 class VarDesc : public VarDescAPI {
  public:
diff --git a/paddle/cinn/frontend/paddle/framework.proto b/paddle/cinn/frontend/paddle/framework.proto
@@ -15,7 +15,7 @@ package cinn.frontend.paddle.proto;
 // Any incompatible changes to ProgramDesc and its dependencies should
 // raise the version defined version.h.
 //
-// Serailization and Deserialization codes should be modified in a way
+// Serialization and Deserialization codes should be modified in a way
 // that supports old versions following the version and compatibility policy.
 message Version { optional int64 version = 1 [ default = 0 ]; }
 
diff --git a/paddle/cinn/frontend/paddle/model_parser.cc b/paddle/cinn/frontend/paddle/model_parser.cc
@@ -74,7 +74,7 @@ void TensorFromStream(std::istream &is,
   tensor->Resize(dims);
   void *buf;
   size_t size = tensor->shape().numel() * SizeOfType(desc.data_type());
-  // alllocate memory
+  // allocate memory
   if (target.arch == Target::Arch::X86) {
     switch (static_cast<int>(desc.data_type())) {
 #define SET_TENSOR(desc, type, precision)     \
diff --git a/paddle/cinn/frontend/pass/cast_collapsing.cc b/paddle/cinn/frontend/pass/cast_collapsing.cc
@@ -210,7 +210,7 @@ class CastCollapsingPass : public ProgramPass {
       if (!can_remove) {
         VLOG(4) << "The cast op " << cast_info
                 << " is useless but fetched, replace with identity.";
-        // cannot remove, but we can replace the cast with indentiy for more
+        // cannot remove, but we can replace the cast with identity for more
         // fusion opportunity
         ReplaceWithIdentity(cast);
       } else {

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ struct CINN_ALIGN(2) bfloat16 {`
`158`	`158`	`return *this;`
`159`	`159`	`}`
`160`	`160`
`161`		`- // Conversion opertors`
	`161`	`+ // Conversion operators`
`162`	`162`	`__host__ __device__ inline operator float() const {`
`163`	`163`	`#ifdef CINN_CUDA_BF16`
`164`	`164`	`return __bfloat162float(reinterpret_cast<const __nv_bfloat16>(&x));`
Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ struct CINN_ALIGN(2) float16 {`
`195`	`195`	`return *this;`
`196`	`196`	`}`
`197`	`197`
`198`		`-// Conversion opertors`
	`198`	`+// Conversion operators`
`199`	`199`	`#ifdef CINN_CUDA_FP16`
`200`	`200`	`__host__ __device__ inline half to_half() const {`
`201`	`201`	`#if CUDA_VERSION >= 9000`