Skip to content

feat(cannon-contracts): MIPS64 conversion#11404

Closed
clabby wants to merge 6 commits intocl/cannon-64bitfrom
cl/cannon-sol-64bit
Closed

feat(cannon-contracts): MIPS64 conversion#11404
clabby wants to merge 6 commits intocl/cannon-64bitfrom
cl/cannon-sol-64bit

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Aug 8, 2024

Overview

Converts MIPS.sol & MIPS2.sol to MIPS64.

Copy link
Contributor Author

clabby commented Aug 8, 2024

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @clabby and the rest of your teammates on Graphite Graphite

nextPC uint64
insn uint32
}{
{"illegal instruction", 0, 0xFF_FF_FF_FF},
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a valid SD instruction, fyi

@clabby clabby force-pushed the cl/cannon-sol-64bit branch 2 times, most recently from 9b6764f to a701d24 Compare August 8, 2024 21:33
@clabby clabby force-pushed the cl/cannon-sol-64bit branch 2 times, most recently from 813bc78 to 21d6ff5 Compare August 9, 2024 15:07
@clabby clabby marked this pull request as ready for review August 9, 2024 15:28
@clabby clabby requested review from a team as code owners August 9, 2024 15:28
@clabby clabby requested review from Inphi, maurelian, mbaxter, protolambda and sebastianst and removed request for a team, maurelian and sebastianst August 9, 2024 15:28
@clabby clabby self-assigned this Aug 9, 2024
@clabby clabby added C-feature Category: features A-cannon Area: cannon A-pkg-contracts-bedrock Area: packages/contracts-bedrock labels Aug 9, 2024
@clabby clabby force-pushed the cl/cannon-sol-64bit branch from 21d6ff5 to a6f8546 Compare August 9, 2024 17:46
@clabby clabby force-pushed the cl/cannon-sol-64bit branch from a6f8546 to a036ea4 Compare August 10, 2024 14:45
@github-actions
Copy link
Contributor

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Aug 25, 2024
@mbaxter mbaxter added S-exempt-stale Status: Do not mark stale and removed Stale labels Aug 29, 2024
@clabby clabby mentioned this pull request Sep 27, 2024
2 tasks
@tynes
Copy link
Contributor

tynes commented Nov 18, 2024

Closing this due to #12029 being merged

@tynes tynes closed this Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cannon Area: cannon A-pkg-contracts-bedrock Area: packages/contracts-bedrock C-feature Category: features S-exempt-stale Status: Do not mark stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants