[clang-tidy] No.52 enable bugprone-argument-comment (#56217)

enkilee · web-flow · commit 0cae0151017c · 2023-08-14T17:42:17.000+08:00
diff --git a/.clang-tidy b/.clang-tidy
@@ -1,6 +1,6 @@
 ---
 Checks: '
--bugprone-argument-comment,
+bugprone-argument-comment,
 -bugprone-assert-side-effect,
 -bugprone-bad-signal-to-kill-thread,
 -bugprone-bool-pointer-implicit-conversion,
diff --git a/paddle/fluid/framework/executor_cache.cc b/paddle/fluid/framework/executor_cache.cc
@@ -266,7 +266,7 @@ CacheInfo GetExecutorInfoFromCache(const ProgramDesc &program_desc,
     auto &cached_value = cached_exe_info.GetMutable(program_id, is_grad);
     cached_value.executor_ = pe_and_graph.first;
     cached_value.graph_ = pe_and_graph.second;
-    return std::make_pair(pe_and_graph.first, /*is_new_created=*/true);
+    return std::make_pair(pe_and_graph.first, true);
   } else {
     VLOG(1) << "get exe_info from cache by: " << program_id
             << " is_grad: " << is_grad;
@@ -280,7 +280,7 @@ CacheInfo GetExecutorInfoFromCache(const ProgramDesc &program_desc,
     // need to recreate tmp variables in new scope
     parallel_executor->PrepareVariables(scope);
 
-    return std::make_pair(parallel_executor, /*is_new_created=*/false);
+    return std::make_pair(parallel_executor, false);
   }
 }
 
diff --git a/paddle/fluid/framework/new_executor/interpreter/stream_analyzer.cc b/paddle/fluid/framework/new_executor/interpreter/stream_analyzer.cc
@@ -58,8 +58,8 @@ void StreamAnalyzer::ConstructEvents(std::vector<Instruction>* instructions) {
     std::vector<std::vector<std::vector<size_t>>> run_type_info(
         instr_num,
         std::vector<std::vector<size_t>>(
-            /*number_of_run_type = */ 2));  // instr_id -> run_type ->
-                                            // next_instr_id
+            /*number_of_run_type = */ 2));  // NOLINT
+    // instr_id -> run_type -> next_instr_id
     AnalyseAllRunType(
         cross_step_merged_instructions_ptr, downstream_map, &run_type_info);
 
diff --git a/paddle/fluid/framework/parallel_executor.cc b/paddle/fluid/framework/parallel_executor.cc
@@ -754,7 +754,7 @@ ParallelExecutor::ParallelExecutor(const platform::Place &place,
                                 /*device_count=*/1,
                                 *graph);
 
-  CreateLocalScopes(scope, /*local_scope=*/{scope}, /*create_new=*/false);
+  CreateLocalScopes(scope, /*local_scopes=*/{scope}, /*create_new=*/false);
 
   // Apply BuildStrategy to compile graph.
   std::vector<ir::Graph *> graphs = {graph};
