Skip to content

ZIR-000: Fix clang-format and rustfmt style violations - #340

Open
zeroecco wants to merge 1 commit into
mainfrom
lint-fix/clang-and-rustfmt-auto
Open

zeroecco wants to merge 1 commit into
mainfrom
lint-fix/clang-and-rustfmt-auto

Conversation

@zeroecco

@zeroecco zeroecco commented May 7, 2026

Copy link
Copy Markdown

Summary

  • Applied clang-format --style=file to 27 C++ source files with style violations
  • Applied rustfmt to 2 Rust files with style violations
  • All changes are non-semantic formatting only (no logic changes)

Files modified

C++ (27 files): zirgen/Dialect/R1CS/IR/Ops.cpp, Types.cpp, zirgen/Dialect/Zll/IR/Codegen.h, zirgen/circuit/bigint/elliptic_curve.h, zirgen/circuit/fib/fib.cpp, zirgen/circuit/recursion/micro.cpp, sha.cpp, test/AB.cpp, wom.cpp, zirgen/circuit/rv32im/v1/edsl/bigint.cpp, bigint2.cpp, body.cpp, compute.cpp, divide.cpp, ecall.cpp, memio.cpp, multiply.cpp, sha.cpp, top.cpp, zirgen/compiler/edsl/component.cpp, zirgen/compiler/picus/picus.cpp, zirgen/components/bytes.cpp, iszero.cpp, mux.h, ram.cpp, u32.cpp, zirgen/dsl/examples/calculator/Fp.h

Rust (2 files): zirgen/dsl/src/cpu.rs, zirgen/dsl/src/adder.rs

Test plan

  • clang-format --dry-run --Werror --style=file passes on all modified C++ files
  • rustfmt --check passes on all modified Rust files

🤖 Generated with Claude Code

Apply automated formatting to 27 C++ files (clang-format --style=file)
and 2 Rust files (rustfmt) to comply with project style configuration.
Also manually fix missing space after comma in cpu.rs use statement
that rustfmt could not resolve due to incomplete file content.

All changes are non-semantic: indentation, line wrapping, and spacing only.

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@zeroecco
zeroecco force-pushed the lint-fix/clang-and-rustfmt-auto branch from 150e032 to d195c13 Compare May 7, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant