Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

fix: Implements handling of the high limb during fixed base scalar multiplication #535

Merged
merged 7 commits into from
Sep 12, 2023
Merged

fix: Implements handling of the high limb during fixed base scalar multiplication #535

merged 7 commits into from
Sep 12, 2023

Conversation

kevaundray
Copy link
Contributor

Description

Continuation of #531 to implement logic to handle the high limb

Problem*

Resolves

Summary*

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@kevaundray @TomAFrench
Update blackbox_solver/src/barretenberg/wasm/scalar_mul.rs
3a20c9e 
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
@kevaundray kevaundray added this pull request to the merge queue Sep 12, 2023
Merged via the queue into master with commit 551504a Sep 12, 2023
15 checks passed
@kevaundray kevaundray mentioned this pull request Sep 12, 2023
Merged
@TomAFrench TomAFrench deleted the kw/add-low-high-test branch September 12, 2023 14:16
TomAFrench added a commit that referenced this pull request Sep 15, 2023
@TomAFrench
Merge branch 'master' into expose-solvers
9f6e8c8 
* master: (40 commits)
  chore: make acvm_js publish trigger on release-please (#550)
  chore(ci): add `cargo deny` workflow (#549)
  fix: use the exact version for the hex crate (#546)
  chore: Release 0.26.1 (#542)
  fix: Implements handling of the high limb during fixed base scalar multiplication (#535)
  chore: fix acvm publishing (#539)
  chore: fix wasm build on macos (#541)
  chore(ci): install `wasm-bindgen-cli` through prebuilt binary (#536)
  chore: add manual trigger to acvm_js publishing (#534)
  chore: acvm_js publishing (#529)
  chore: Release 0.26.0 (#533)
  chore!: Add a low and high limb to scalar mul opcode (#532)
  chore: Release 0.25.0 (#526)
  fix: initialize recursive proof output to zero (#524)
  chore(ci): update to actions/checkout@v4 (#525)
  feat!: Provide runtime callstacks for brillig failures and return errors in acvm_js (#523)
  chore: add ci for acvmjs rust test (#516)
  chore: Release 0.24.1 (#519)
  fix: Add WASI 20 `_initialize` call to `acvm_backend.wasm` binary (#518)
  chore: Release 0.24.0 (#512)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@TomAFrench TomAFrench TomAFrench approved these changes

@sirasistant sirasistant sirasistant approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kevaundray @sirasistant @TomAFrench