Repository navigation

Native tooling: format every file, clang-tidy by family, stronger warnings, drop unused build options - #762

Merged

jtravs merged 92 commits into

mainfrom

chore/native-tooling

Nov 12, 2026

Merged

Native tooling: format every file, clang-tidy by family, stronger warnings, drop unused build options#762
jtravs merged 92 commits into
mainfrom
chore/native-tooling

Commits on Nov 11, 2026
- Keep only the hybrid GEMM provider
  
  jtravs
  and
  claude
  committed
  bd8cd8b
- Remove the GEMM microbenchmark
  
  jtravs
  and
  claude
  committed
  19c17b8
- Drop IntelLLVM (icpx) support
  
  jtravs
  and
  claude
  committed
  f5d4234
- Remove the Kokkos Threads host backend option
  
  jtravs
  and
  claude
  committed
  e04648a
- Build and link no single-precision FFTW
  
  jtravs
  and
  claude
  committed
  f415464
- Fold the native benchmark scripts into benchmarks/
  
  jtravs
  and
  claude
  committed
  5d2539a
- Format every tracked native C++ file and regroup includes
  
  jtravs
  and
  claude
  committed
  836b93b
- Leave clang-format's RemoveSemicolon off
  
  jtravs
  and
  claude
  committed
  5145dde
- Format the native C++ files
  
  jtravs
  and
  claude
  committed
  e8a55f7
- Pin clang-tidy to LLVM 23.1 and add poe tidy-native
  
  jtravs
  and
  claude
  committed
  ae2804b
- Add stronger warnings, listed once
  
  jtravs
  and
  claude
  committed
  77be8eb
- Stop the sanitizers at the first report and check view bounds
  
  jtravs
  and
  claude
  committed
  5202370
- Add a weekly native deep-check workflow; widen compute-sanitizer
  
  jtravs
  and
  claude
  committed
  c7bf92a
- Record the removed native build options
  
  jtravs
  and
  claude
  committed
  e9462fa
- Build SLEEF's header before tidying outside the build
  
  jtravs
  and
  claude
  committed
  06fe345
- Refuse a single-value read of a multi-valued stage table
  
  jtravs
  and
  claude
  committed
  3e6b3fa
- Write 1/1 as 1.0 in the Adams divisor tables
  
  jtravs
  and
  claude
  committed
  2a10abd
- Hold FFTW plan handles as auto* const
  
  jtravs
  and
  claude
  committed
  2abdefc
- Keep clang-format from reflowing NOLINT comments
  
  jtravs
  and
  claude
  committed
  02501e5
- Declare every special member of the RAII classes
  
  jtravs
  and
  claude
  committed
  a35d19f
- Read optionals the checks cannot prove set through .value()
  
  jtravs
  and
  claude
  committed
  7aeb499
- Name the nanobind exception registrations
  
  jtravs
  and
  claude
  committed
  63ae8b3
- Format the exception registrations
  
  jtravs
  and
  claude
  committed
  e15f573
- Discard unique_ptr::release explicitly where a capsule owns the storage
  
  jtravs
  and
  claude
  committed
  5464677
- Enable clang-tidy by whole check families
  
  jtravs
  and
  claude
  committed
  e87eb62
- Document the native tooling: what is adopted, scheduled and rejected
  
  jtravs
  and
  claude
  committed
  fa66a56
- Use #ifdef and #ifndef for single-macro conditions
  
  jtravs
  and
  claude
  committed
  572f890
- Declare one variable per statement
  
  jtravs
  and
  claude
  committed
  f075292
- Mark value-returning accessors [[nodiscard]]
  
  jtravs
  and
  claude
  committed
  3251c0e
- Initialise aggregates with designated initialisers
  
  jtravs
  and
  claude
  committed
  08b7a8c
- Call the std::ranges algorithms on whole containers
  
  jtravs
  and
  claude
  committed
  eef1956
- Copy the Adams rate weights from the vector, not its pointer
  
  jtravs
  and
  claude
  committed
  ebf37d1
- Leave misc-const-correctness off
  
  jtravs
  and
  claude
  committed
  007ed55
- Drop the empty parameter lists of lambdas that take none
  
  jtravs
  and
  claude
  committed
  e3cf938
- Brace both arms of an if-else when one needs braces
  
  jtravs
  and
  claude
  committed
  3928606
Commits on Nov 12, 2026
- Keep readability-simplify-boolean-expr from applying De Morgan
  
  jtravs
  and
  claude
  committed
  61659b0
- Spell cast-initialised locals with auto
  
  jtravs
  and
  claude
  committed
  42ded57
- Qualify auto pointers as auto*
  
  jtravs
  and
  claude
  committed
  86b97da
- Return braced initialisers where the type is the return type
  
  jtravs
  and
  claude
  committed
  e24a6a1
- Leave readability-use-std-min-max off
  
  jtravs
  and
  claude
  committed
  5cc4203
- Compare the build record's 0/1 macros with zero
  
  jtravs
  and
  claude
  committed
  8126da6
- Call void functions and return separately in the band filter dispatch
  
  jtravs
  and
  claude
  committed
  998f648
- Use std::numbers for sqrt(2) and ln(10)
  
  jtravs
  and
  claude
  committed
  1db5de8
- Take execution spaces by value where they are stored
  
  jtravs
  and
  claude
  committed
  939ed00
- Remove duplicate includes
  
  jtravs
  and
  claude
  committed
  36643cf
- Include <numbers> where std::numbers is used
  
  jtravs
  and
  claude
  committed
  584f15b
- Compare mixed-sign integers with std::cmp_less
  
  jtravs
  and
  claude
  committed
  68e377d
- Name declarations' parameters as their definitions do
  
  jtravs
  and
  claude
  committed
  dcab1c2
- Name the unused parameter
  
  jtravs
  and
  claude
  committed
  51819a2
- Return the GEMM shape policy literal as a string_view
  
  jtravs
  and
  claude
  committed
  a9b02a3
- Write a message with quotes as a raw string literal
  
  jtravs
  and
  claude
  committed
  dbbbbf9
- Move each session event into its batch
  
  jtravs
  and
  claude
  committed
  52554d1
- Reserve the test vectors filled in loops
  
  jtravs
  and
  claude
  committed
  87f3cca
- Give general.cpp's file-local helpers internal linkage
  
  jtravs
  and
  claude
  committed
  024a118
- Let Kokkos views pass by value under performance-unnecessary-value-param
  
  jtravs
  and
  claude
  committed
  b0753a6
- Defer the bindings' by-value parameters to their rewrite
  
  jtravs
  and
  claude
  committed
  3a8e517
- Leave readability-convert-member-functions-to-static off
  
  jtravs
  and
  claude
  committed
  0492fca
- Mark the far field's packed-row cascade for bugprone-assignment-in-if-condition
  
  jtravs
  and
  claude
  committed
  f08863c
- Keep the tests' fixed random seeds
  
  jtravs
  and
  claude
  committed
  41a9f5e
- Import solprop_native lazily in benchmarks.statistics_cost
  
  jtravs
  and
  claude
  committed
  16e435b
- Let clang-tidy ignore GCC-only warning flags
  
  jtravs
  and
  claude
  committed
  2f34dc3
- Keep the Adams lane passes' integer masks
  
  jtravs
  and
  claude
  committed
  3fd07c1
- Mark the deliberate argument orders readability-suspicious-call-argument flags
  
  jtravs
  and
  claude
  committed
  47f7e6f
- Release the closed session's rows without a throwing assignment
  
  jtravs
  and
  claude
  committed
  6a1ff0b
- Form size products in std::size_t
  
  jtravs
  and
  claude
  committed
  a988876
- Name the attempt rows' count before converting it
  
  jtravs
  and
  claude
  committed
  a86456f
- Build the coupled-flow test's tables without throwing static initialisers
  
  jtravs
  and
  claude
  committed
  47afb12
- Test the planned wisdom lines with std::ranges::any_of
  
  jtravs
  and
  claude
  committed
  367841e
- Keep the noexcept workspace helpers from allocating
  
  jtravs
  and
  claude
  committed
  6b7eb29
- Put column_norms in an anonymous namespace
  
  jtravs
  and
  claude
  committed
  5f0abdd
- Declare the Raman task counts one per statement
  
  jtravs
  and
  claude
  committed
  14006e2
- Test the active mask's bytes against zero
  
  jtravs
  and
  claude
  committed
  51abda3
- Name the integer quotients that feed floating-point values
  
  jtravs
  and
  claude
  committed
  eb3ad41
- Form general.cpp's size constants in std::size_t
  
  jtravs
  and
  claude
  committed
  a6c9a38
- Flatten the envelope BLAS guard in the transform policies
  
  jtravs
  and
  claude
  committed
  15991c4
- Close the band statistics' host-only else arms inside their #if
  
  jtravs
  and
  claude
  committed
  0b42a12
- Mark general.cpp's coincidental argument names
  
  jtravs
  and
  claude
  committed
  864ff6b
- Convert Kokkos's accessibility enum to bool explicitly
  
  jtravs
  and
  claude
  committed
  005d5ae
- Leave the analyser's NewDeleteLeaks check off
  
  jtravs
  and
  claude
  committed
  ccc28d7
- Mark the generated source digest's external linkage
  
  jtravs
  and
  claude
  committed
  a80e3f1
- Install the pinned clang-tidy on Linux and macOS only
  
  jtravs
  and
  claude
  committed
  b4ac85b
- Time the statistics channel on the former harness's CtC workload
  
  jtravs
  and
  claude
  committed
  3eb5bcd
- Run poe tidy-native with the pinned clang-tidy the build resolved
  
  jtravs
  and
  claude
  committed
  4917127
- Refuse multi-valued scalar stage tables when they are loaded
  
  jtravs
  and
  claude
  committed
  f38c72e
- Allow nanobind handles by value instead of a file-wide NOLINT
  
  jtravs
  and
  claude
  committed
  a450309
- Find only FFTW's double libraries before Kokkos-FFT
  
  jtravs
  and
  claude
  committed
  0fdcf73
- Let the weekly report-only builds warn without failing
  
  jtravs
  and
  claude
  committed
  5a8eb58
- List the pinned clang-format and clang-tidy in the dependencies
  
  jtravs
  and
  claude
  committed
  6fc670e
- Run native-deep.yml on a pull request that changes it
  
  jtravs
  and
  claude
  committed
  3a18591
- Merge origin/main into chore/native-tooling
  
  jtravs
  and
  claude
  committed
  f9d6b99
- Take statistics_cost's run options by keyword
  
  jtravs
  and
  claude
  committed
  2025943
- Preload libstdc++ with ASan in the weekly extension job
  
  jtravs
  and
  claude
  committed
  4393ad8

Commits

Commits on Nov 11, 2026

Commits on Nov 12, 2026