• Lei Wang's avatar
    [Lint] Introduce clang-tidy into format.sh (#777) · cdc5d8d3
    Lei Wang authored
    * [Refactor] Update Clang-Tidy Checks and Improve Code Consistency
    
    - Enhanced .clang-tidy configuration by adding specific checks for better bug detection and performance optimization.
    - Refactored function signatures across multiple files to use `const` references for parameters, improving performance and code clarity.
    - Updated various methods to ensure consistent handling of parameters, particularly in `AddPredicate`, `Substitute`, and `PlanLoopPartition` functions.
    - Improved readability by replacing size checks with `empty()` method calls in several locations, ensuring clearer intent in the code.
    - General code cleanup and adherence to best practices for better maintainability.
    
    * [Refactor] Enhance Code Consistency and Clang-Tidy Configuration
    
    - Updated .clang-tidy configuration to include additional checks for improved code quality and performance.
    - Refactored function signatures across multiple files to use `const` references, enhancing performance and clarity.
    - Replaced size checks with `empty()` method calls in various locations for clearer intent.
    - Improved handling of parameters in several functions, ensuring consistent usage of `std::move` where applicable.
    - General code cleanup to adhere to best practices and improve maintainability.
    
    * [Refactor] Integrate Clang-Tidy Checks and Enhance Code Consistency
    
    - Added clang-tidy checks to the format script for improved code quality assurance.
    - Refactored function signatures across multiple files to consistently use `const` references, enhancing performance and clarity.
    - Updated the requirements-lint.txt file to include clang-tidy as a dependency.
    - General code cleanup to adhere to best practices and improve maintainability.
    
    * [CI] Update AMD CI Workflow to Include Build Directory Creation
    
    - Added steps to create a build directory and configure CMake with ROCm support during the format check process.
    - Ensured cleanup of the build directory after the format check to maintain a clean workspace.
    
    * [Refactor] Remove Unused Member Variables in AtomicAddNode and CopyNode
    
    - Removed the `args_` member variable from both `AtomicAddNode` and `CopyNode` classes to streamline the code and eliminate unnecessary data members.
    - This change enhances code clarity and maintainability by focusing on relevant attributes for each class.
    
    * [Refactor] Update Clang-Tidy Integration and Code Improvements
    
    - Modified the format script to include the `-fix` option in the clang-tidy command for automatic code fixes.
    - Refactored the `AtomicAddVectorizePlanner` class to improve variable handling and consistency, including changes to member variable types and function signatures.
    - Enhanced code clarity by removing unnecessary `std::move` calls and ensuring consistent usage of types across the class.
    - General code cleanup to adhere to best practices and improve maintainability.
    
    * [Refactor] Improve Parameter Handling and Consistency in AtomicAddVectorize
    
    - Updated function signatures in `AtomicAddVectorizePlanResult` and `AtomicAddVectorizeRewriter` to use `const` references and `std::move` for better performance and clarity.
    - Enhanced the `UpdateVectorSize` method to accept `const Array<PrimExpr>&` for improved efficiency.
    - General code cleanup to maintain consistency and adhere to best practices.
    
    * [CI] Add Git Submodule Initialization to CI Workflow
    
    - Included a step to initialize and update git submodules recursively in the CI workflow.
    - This change ensures that all necessary submodules are available during the format check process, improving build reliability.
    
    * [CI] Add Git Submodule Update Step to Format Check
    
    - Included a command to initialize and update git submodules recursively in the CI workflow during the format check process.
    - This enhancement ensures that all required submodules are available, contributing to improved build reliability.
    
    * [Refactor] Update Function Signatures in AtomicAddVectorize
    
    - Modified the `VectorizeAtomicAdd` function signature to use `const` references for `thread_var` and `thread_bounds`, enhancing performance and code clarity.
    - This change aligns with previous refactoring efforts to improve parameter handling and consistency across the codebase.
    cdc5d8d3
ci.yml 4.35 KB