fix comments

DeepLink-org · Dec 8, 2023 · 165dd81 · 165dd81
1 parent c158a57
commit 165dd81
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/dipu/scripts/autogen_diopi_wrapper/diopi_functions.yaml b/dipu/scripts/autogen_diopi_wrapper/diopi_functions.yaml
@@ -1598,8 +1598,8 @@
     *   '''
     */
 
-    int SameAsInputSize = 7;
-    std::vector<at::Tensor> outputs(SameAsInputSize);
+    constexpr int kSameAsInputSize = 7;
+    std::vector<at::Tensor> outputs(kSameAsInputSize);
     outputs[0] = result;
     return outputs;
 
@@ -1709,8 +1709,8 @@
     *   '''
     */
     
-    int SameAsInputSize = 7;
-    std::vector<at::Tensor> outputs(SameAsInputSize);
+    constexpr int kSameAsInputSize = 7;
+    std::vector<at::Tensor> outputs(kSameAsInputSize);
     outputs[0] = result;
     return outputs;
 

diff --git a/dipu/scripts/autogen_diopi_wrapper/diopi_wrapper_template.py b/dipu/scripts/autogen_diopi_wrapper/diopi_wrapper_template.py
@@ -20,7 +20,7 @@
 
 $header_include_code
 
-// NOTS: some kernels(eg. _foreach_add_.List) have custom code at the beginning with direct return, cause this warning
+// NOTE: some kernels (e.g. _foreach_add_.List) have custom codes at the beginning ending with early return.
 // NOLINTBEGIN(readability-redundant-control-flow)
 
 namespace dipu {