Skip to content

Add Venom consecutive ADD folding pass#717

Draft
augmentcode[bot] wants to merge 1 commit into
mainfrom
michaelos443/venom-consecutive-add-folding
Draft

Add Venom consecutive ADD folding pass#717
augmentcode[bot] wants to merge 1 commit into
mainfrom
michaelos443/venom-consecutive-add-folding

Conversation

@augmentcode
Copy link
Copy Markdown

@augmentcode augmentcode Bot commented May 27, 2026

Summary

  • add ConsecutiveAddFoldingPass for single-use chained add instructions with compile-time constant operands
  • wire the pass into O2/O3/Os pipelines and the algebraic-optimization disable flag
  • add focused Venom pre/post tests for literal, assigned-constant, three-deep, zero-canceling, and multi-use cases

Tests

  • SETUPTOOLS_SCM_PRETEND_VERSION=0.0.0 uv run pytest tests/unit/compiler/venom/test_consecutive_add_folding.py tests/unit/compiler/venom/test_algebraic_optimizer.py
  • SETUPTOOLS_SCM_PRETEND_VERSION=0.0.0 uv run python - <<'PY' ... pass order validation ... PY
  • SETUPTOOLS_SCM_PRETEND_VERSION=0.0.0 uv run black --check vyper/venom/passes/consecutive_add_folding.py tests/unit/compiler/venom/test_consecutive_add_folding.py

Notes

  • This is a manual session, but the required github-api tool is not available here, so this PR was created with github-app-api.

Pull Request opened by Augment Code | View session

@augmentcode
Copy link
Copy Markdown
Author

augmentcode Bot commented May 27, 2026

PR Author Agent

👋 I've got this PR — here's what I'll handle for you:

  • Review feedback — implement suggestions, answer questions, fix what comes up
  • CI failures — I get pinged when checks fail and try to fix them
  • Merge conflicts — bring the PR back up to date when it falls behind
  • Merge gates — watch CI, reviews, and verification
  • Ping you the moment it's ready to merge

Marking it ready and picking reviewers are your call — I'll leave both alone.

Drop a comment anytime!

@augmentcode
Copy link
Copy Markdown
Author

augmentcode Bot commented May 27, 2026

PR Author Agent

All blocking gates are green on 7f5cb257d13160127e362736d89367cf9b24d790 — ready to flip to ready-for-review whenever you are. Reply mark ready here, or use the Ready for review button on GitHub.

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