Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Different compiled bytecode for ecdsa_account_contract.json on m1 vs x86_64 #2959

Closed
dan-aztec opened this issue Oct 3, 2023 · 2 comments
Closed
Assignees
Labels
aztec.nr Helpful for development of Aztec.nr the smart contract framework bug Something isn't working

Comments

@dan-aztec
Copy link

Aim

In the AztecProtocol monorepo, I'm getting different bytecode generated for yarn-project/aztec.js/src/abis/ecdsa_account_contract.json when I compile/build locally on m1 vs mainframe.

Not sure why this is the only contract where the abi/artifact is different.

noticed this on my branch (which doesnt change the noir source).

Expected Behavior

the same bytecode to be produced in yarn-project/aztec.js/src/abis/ecdsa_account_contract.json on m1 and x_86

Bug

just slightly different bytecode in a single aztec contract

To Reproduce

  1. in aztec-packages, checkout dan/test-nargo-12 (or master, as of October 3rd 2023)
  2. do top level ./bootstrap.sh or within yarn-projects/noir-contracts do ./boostrap.sh
  3. git status will show yarn-project/aztec.js/src/abis/ecdsa_account_contract.json has changed on m1

Installation Method

Binary

Nargo Version

nargo 0.12.0 (git version hash: 3b43709, is dirty: false)

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

@dan-aztec dan-aztec added the bug Something isn't working label Oct 3, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Oct 3, 2023
@guipublic
Copy link
Contributor

I did the same on my M1 and on the mainframe, from master, and I get the same result.
How can I ensure there were properly generated by the bootstrap?

@Savio-Sou Savio-Sou added the aztec.nr Helpful for development of Aztec.nr the smart contract framework label Oct 6, 2023
@kevaundray
Copy link
Contributor

Closing as this is not reproducable

@kevaundray kevaundray closed this as not planned Won't fix, can't repro, duplicate, stale Jan 15, 2024
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Noir Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aztec.nr Helpful for development of Aztec.nr the smart contract framework bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

4 participants