Skip to content

ci: add scalar fallback fuzzing targets - #537

Merged
hellobertrand merged 1 commit into
mainfrom
ci/fuzz-scalar-path
Oct 19, 2026
Merged

hellobertrand merged 1 commit into
mainfrom
ci/fuzz-scalar-path

Conversation

@hellobertrand

@hellobertrand hellobertrand commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Allows fuzzers to verify correctness on platforms without SIMD support, such as WASM, RISC-V, s390x, and ppc.

Summary by CodeRabbit

  • Tests
    • Added a scalar-mode build of the roundtrip fuzzer alongside the existing default fuzzer builds.
  • Documentation
    • Updated the documented fuzzing workflow triggers: pull requests and manual runs are listed, while scheduled runs are no longer listed.

@hellobertrand hellobertrand added the github_actions Pull requests that update GitHub Actions code label Oct 18, 2026
@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 443fbbbf-c52f-4e89-a1e6-b3eda955b56d
📥 Commits

Reviewing files that changed from the base of the PR and between fb3fe4a and 78ed3ad.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1a4b3981-367e-4848-9bdc-f8321186e2d7
📥 Commits

Reviewing files that changed from the base of the PR and between b9204c6 and fb3fe4a.

📒 Files selected for processing: 2
  • .clusterfuzzlite/build.sh
  • .github/workflows/README.md

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review.

📝 Walkthrough

The fuzz build script now creates per-build object directories and adds a SIMD-disabled roundtrip output. The workflow README no longer lists a scheduled trigger every three days.

Changes: Fuzzer build outputs

Layer / File(s) Summary
Default and scalar fuzzer builds
.clusterfuzzlite/build.sh
A parameterized build function compiles library and fuzzer objects into per-build directories. The script builds the existing fuzzer set and adds a SIMD-disabled roundtrip output with the _scalar suffix.

Workflow trigger documentation

Layer / File(s) Summary
Documented workflow triggers
.github/workflows/README.md
The documented fuzzing.yml triggers retain pull requests and manual dispatch and remove the scheduled run every three days.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to fb3fe

The PR adds a scalar roundtrip fuzz target without changing workflow triggers; no actionable merge risk is established.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: adding scalar fallback fuzzing targets in CI.
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 …
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Introduce a build helper to support testing both standard and scalar-only code paths. This allows the fuzzer to verify correctness on platforms without SIMD support, such as WASM, RISC-V, s390x, and ppc.

Signed-off-by: Bertrand Lebonnois <5901952+hellobertrand@users.noreply.github.com>
@hellobertrand
hellobertrand merged commit 00b0f65 into main Oct 19, 2026
94 checks passed
@hellobertrand
hellobertrand deleted the ci/fuzz-scalar-path branch October 19, 2026 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant