Upgrade clang-format to v22 and improve pylint pre-commit configuration (#2813)

## Summary

- Migrate clang-format from `pocc/pre-commit-hooks` to
`pre-commit/mirrors-clang-format` v22.1.0
- Update `.clang-format` config to maintain consistent code style across
versions
- Configure pylint to properly handle C extension modules (numpy, dpctl)
- Remove manual clang-format-12 installation from GitHub workflow
- Reformat all C++ files with clang-format v22
- Add reformatting commit to `.git-blame-ignore-revs`

## Changes

### Pre-commit configuration
- Switch to `pre-commit/mirrors-clang-format` (v22.1.0) for better
version pinning and consistency across environments
- Add `--disable=c-extension-no-member` to pylint to avoid false
positives on C extension objects

### Code style
- Update `.clang-format`: Set `AfterControlStatement: Never` to preserve
existing brace placement style
- Reformat all C++ source files with clang-format v22 (net -159 lines
due to template parameter formatting improvements)

### Pylint configuration
- Add `extension-pkg-allow-list = ["numpy"]` to help pylint understand
numpy better
- Add `generated-members` patterns to handle dynamically created
attributes on numpy types

### CI/CD
- Remove obsolete `apt-get install clang-format-12` step from pre-commit
workflow

Author: antonwolfy

.clang-format

@@ -16,7 +16,7 @@ BinPackParameters: false
 BraceWrapping:
   AfterCaseLabel:  true
   AfterClass:      true
-  AfterControlStatement: MultiLine
+  AfterControlStatement: Never
   AfterEnum:       true
   AfterFunction:   true
   AfterNamespace:  true

.git-blame-ignore-revs

@@ -17,3 +17,6 @@ c106d91b866f4acd30226b68519b12a73a881490
 
 # Add pygrep-hooks to pre-commit config
 e62718415aa3660da5f607e352c991a063a54219
+
+# Bump clang-format from 12.0.1 to 22.1.0 version
+c2d65bd451a7d8e5b6319147da95e9dabf7a382b

.github/workflows/pre-commit.yml

@@ -15,13 +15,6 @@ jobs:
     timeout-minutes: 10
 
     steps:
-      - name: Set up clang-format
-        run: |
-          sudo apt-get install -y clang-format-12
-          sudo unlink /usr/bin/clang-format
-          sudo ln -s /usr/bin/clang-format-12 /usr/bin/clang-format
-          clang-format --version
-
       - name: Set up pip packages
         uses: BSFishy/pip-action@8f2d471d809dc20b6ada98c91910b6ae6243f318 # v1
         with:

.pre-commit-config.yaml

@@ -88,8 +88,8 @@ repos:
         additional_dependencies:
             - flake8-docstrings==1.7.0
             - flake8-bugbear==24.12.12
--   repo: https://github.com/pocc/pre-commit-hooks
-    rev: v1.3.5
+-   repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: v22.1.0
     hooks:
     -   id: clang-format
         args: ["-i"]
@@ -114,7 +114,8 @@ repos:
             "-sn", # Don't display the score
             "--disable=import-error",
             "--disable=redefined-builtin",
-            "--disable=unused-wildcard-import"
+            "--disable=unused-wildcard-import",
+            "--disable=c-extension-no-member"
             ]
         files: '^dpnp/(dpnp_iface.*|fft|linalg|scipy|dpnp_array)'
 -   repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks

benchmarks/asv.conf.json

@@ -15,7 +15,9 @@
 
     // List of branches to benchmark. If not provided, defaults to "master"
     // (for git) or "tip" (for mercurial).
-    "branches": ["HEAD"],
+    "branches": [
+        "HEAD"
+    ],
 
     // The DVCS being used.  If not set, it will be automatically
     // determined from "repo" by looking at the protocol in the URL
@@ -35,7 +37,9 @@
 
     // The Pythons you'd like to test against.  If not provided, defaults
     // to the current version of Python used to run `asv`.
-    "pythons": ["3.7"],
+    "pythons": [
+        "3.7"
+    ],
 
     // The matrix of dependencies to test.  Each key is the name of a
     // package (in PyPI) and the values are version numbers.  An empty
@@ -53,7 +57,6 @@
     // environments in.  If not provided, defaults to "env"
     "env_dir": "env",
 
-
     // The directory (relative to the current directory) that raw benchmark
     // results are stored in.  If not provided, defaults to "results".
     "results_dir": "results",
@@ -79,7 +82,8 @@
     // skipped for the matching benchmark.
     //
     // "regressions_first_commits": {
-    //    "some_benchmark": "352cdf",  // Consider regressions only after this commit
+    //    "some_benchmark": "352cdf",  // Consider regressions only after this
+    //    commit
     //    "another_benchmark": null,   // Skip regression detection altogether
     // }
 }

dpnp/backend/extensions/blas/dot_common.hpp

@@ -97,8 +97,7 @@ std::pair<sycl::event, sycl::event>
 
     if (!dpctl::utils::queues_are_compatible(
             exec_q,
-            {vectorX.get_queue(), vectorY.get_queue(), result.get_queue()}))
-    {
+            {vectorX.get_queue(), vectorY.get_queue(), result.get_queue()})) {
         throw py::value_error(
             "USM allocations are not compatible with the execution queue.");
     }
@@ -120,8 +119,8 @@ std::pair<sycl::event, sycl::event>
     const int vectorY_typenum = vectorY.get_typenum();
     const int result_typenum = result.get_typenum();
 
-    if (result_typenum != vectorX_typenum || result_typenum != vectorY_typenum)
-    {
+    if (result_typenum != vectorX_typenum ||
+        result_typenum != vectorY_typenum) {
         throw py::value_error("Given arrays must be of the same type.");
     }
 

dpnp/backend/extensions/blas/gemm.cpp

@@ -181,8 +181,7 @@ std::tuple<sycl::event, sycl::event, bool>
 
     if (!dpctl::utils::queues_are_compatible(
             exec_q,
-            {matrixA.get_queue(), matrixB.get_queue(), resultC.get_queue()}))
-    {
+            {matrixA.get_queue(), matrixB.get_queue(), resultC.get_queue()})) {
         throw py::value_error(
             "USM allocations are not compatible with the execution queue.");
     }

dpnp/backend/extensions/blas/gemm_batch.cpp

@@ -237,8 +237,7 @@ std::tuple<sycl::event, sycl::event, bool>
 
     if (!dpctl::utils::queues_are_compatible(
             exec_q,
-            {matrixA.get_queue(), matrixB.get_queue(), resultC.get_queue()}))
-    {
+            {matrixA.get_queue(), matrixB.get_queue(), resultC.get_queue()})) {
         throw py::value_error(
             "USM allocations are not compatible with the execution queue.");
     }

dpnp/backend/extensions/blas/gemv.cpp

@@ -169,8 +169,7 @@ std::pair<sycl::event, sycl::event>
 
     if (!dpctl::utils::queues_are_compatible(
             exec_q,
-            {matrixA.get_queue(), vectorX.get_queue(), vectorY.get_queue()}))
-    {
+            {matrixA.get_queue(), vectorX.get_queue(), vectorY.get_queue()})) {
         throw py::value_error(
             "USM allocations are not compatible with the execution queue.");
     }

dpnp/backend/extensions/blas/syrk.cpp

@@ -248,8 +248,7 @@ std::pair<sycl::event, sycl::event>
     }
 
     if (!dpctl::utils::queues_are_compatible(
-            exec_q, {matrixA.get_queue(), resultC.get_queue()}))
-    {
+            exec_q, {matrixA.get_queue(), resultC.get_queue()})) {
         throw py::value_error(
             "USM allocations are not compatible with the execution queue.");
     }