From 3e4c4d914d803c18cfd3c549ad3c92d983609c94 Mon Sep 17 00:00:00 2001 From: gouzil <66515297+gouzil@users.noreply.github.com> Date: Sat, 1 Jul 2023 20:43:08 +0800 Subject: [PATCH] [CodeStyle][CINN] fix cinn cpplint codestyle 50-55 (#55021) --- paddle/cinn/common/axis.cc | 2 +- paddle/cinn/hlir/op/elementwise.cc | 6 +++--- paddle/cinn/hlir/pass/check_fusion_accuracy_pass.cc | 4 ++-- paddle/cinn/pybind/frontend.cc | 1 - paddle/cinn/pybind/runtime.cc | 4 ++-- paddle/cinn/runtime/cuda/cuda_util.cc | 3 ++- 6 files changed, 10 insertions(+), 10 deletions(-) diff --git a/paddle/cinn/common/axis.cc b/paddle/cinn/common/axis.cc index bb1d45e9c23..0e13c6bd078 100644 --- a/paddle/cinn/common/axis.cc +++ b/paddle/cinn/common/axis.cc @@ -23,7 +23,7 @@ namespace cinn { namespace common { -const static std::vector kAxises({ +static const std::vector kAxises({ "i", // level 0 "j", // level 1 "k", // level 2 diff --git a/paddle/cinn/hlir/op/elementwise.cc b/paddle/cinn/hlir/op/elementwise.cc index c2ac4e1a1b2..04c89093de3 100644 --- a/paddle/cinn/hlir/op/elementwise.cc +++ b/paddle/cinn/hlir/op/elementwise.cc @@ -476,7 +476,7 @@ std::shared_ptr StrategyForAssignValue( absl::get>(value), out_type[0], tensor_name); \ } - if (false) { + if (false) { // NOLINT } EXPAND_ATTR_TYPE(EXPAND_VALUE_TO_TENSOR) else { @@ -517,7 +517,7 @@ std::vector InferShapeForAssignValue( shape.emplace_back(absl::get>(value).size()); \ } - if (false) { + if (false) { // NOLINT } EXPAND_ATTR_TYPE(EXPAND_ATTR_TO_GET_SHAPE) else { @@ -558,7 +558,7 @@ std::vector InferDtypeForAssignValue( out_type = common::type_of(); \ } - if (false) { + if (false) { // NOLINT } EXPAND_ATTR_TYPE(EXPAND_ATTR_TO_GET_DTYPE) else { diff --git a/paddle/cinn/hlir/pass/check_fusion_accuracy_pass.cc b/paddle/cinn/hlir/pass/check_fusion_accuracy_pass.cc index c2d6249c28a..3573adb8101 100644 --- a/paddle/cinn/hlir/pass/check_fusion_accuracy_pass.cc +++ b/paddle/cinn/hlir/pass/check_fusion_accuracy_pass.cc @@ -80,8 +80,8 @@ class AssertMsg { class CheckFusionAccuracyPass { public: - CheckFusionAccuracyPass(Graph* graph_) - : graph_(graph_), + CheckFusionAccuracyPass(Graph* graph) + : graph_(graph), shape_dict_(graph_->GetMutableAttrs("infershape")), dtype_dict_(graph_->GetMutableAttrs("inferdtype")) {} diff --git a/paddle/cinn/pybind/frontend.cc b/paddle/cinn/pybind/frontend.cc index dcddbd47e27..9c55ef65a80 100644 --- a/paddle/cinn/pybind/frontend.cc +++ b/paddle/cinn/pybind/frontend.cc @@ -935,7 +935,6 @@ void BindFrontend(pybind11::module *m) { << " in CINN! Please check."; return self.var_model_to_program_map().at(paddle_name); }); - } // namespace frontend #undef EXPAND_CINN_SUPPORT_TYPE diff --git a/paddle/cinn/pybind/runtime.cc b/paddle/cinn/pybind/runtime.cc index 9d562298e5b..c69513a2503 100644 --- a/paddle/cinn/pybind/runtime.cc +++ b/paddle/cinn/pybind/runtime.cc @@ -193,7 +193,7 @@ void BindCinnRuntime(py::module *m) { .def_readwrite("flag", &cinn_buffer_t::flag) .def_readwrite("type", &cinn_buffer_t::type) .def_readwrite("dimensions", &cinn_buffer_t::dimensions) - //.def_readwrite("dims", &cinn_buffer_t::dims) + // .def_readwrite("dims", &cinn_buffer_t::dims) .def_readwrite("lazy", &cinn_buffer_t::lazy) .def_readwrite("memory_size", &cinn_buffer_t::memory_size) .def_readwrite("align", &cinn_buffer_t::align) @@ -228,7 +228,7 @@ void BindCinnRuntime(py::module *m) { m->def("cinn_x86_device_interface", &cinn_x86_device_interface) .def("cinn_buffer_load_float32", &cinn_buffer_load_float32) .def("cinn_buffer_load_float64", &cinn_buffer_load_float64); - //.def("cinn_buffer_slice", &cinn_buffer_slice, + // .def("cinn_buffer_slice", &cinn_buffer_slice, // py::return_value_policy::reference); py::class_ cinn_value(*m, "cinn_value_t"); diff --git a/paddle/cinn/runtime/cuda/cuda_util.cc b/paddle/cinn/runtime/cuda/cuda_util.cc index e63a5d70883..a753db58561 100644 --- a/paddle/cinn/runtime/cuda/cuda_util.cc +++ b/paddle/cinn/runtime/cuda/cuda_util.cc @@ -101,7 +101,8 @@ void cinn_call_cuda_kernel(void *kernel_fn, cinn_pod_value_t *args = static_cast(v_args); for (int idx = 0; idx < num_args; ++idx) { if (args[idx].type_code() == ::cinn_type_code()) { - kernel_args.emplace_back(&((cinn_buffer_t *)(args[idx]))->memory); + kernel_args.emplace_back( + &((cinn_buffer_t *)(args[idx]))->memory); // NOLINT } else { kernel_args.emplace_back(args[idx].data_addr()); } -- GitLab