Skip to content

feat: update REVERT_ROOT_HASH_ADDRESS#261

Open
stephenctw wants to merge 1 commit intofeature/bump-emulator-2-stephenfrom
feature/update-checkpoint-address
Open

feat: update REVERT_ROOT_HASH_ADDRESS#261
stephenctw wants to merge 1 commit intofeature/bump-emulator-2-stephenfrom
feature/update-checkpoint-address

Conversation

@stephenctw
Copy link
Copy Markdown
Collaborator

@stephenctw stephenctw commented Apr 10, 2026

Resolving checkpoint address change mentioned here.

@stephenctw stephenctw requested a review from edubart April 10, 2026 00:12
@stephenctw stephenctw self-assigned this Apr 10, 2026
@stephenctw stephenctw changed the base branch from main to feature/bump-emulator-2-stephen April 10, 2026 01:44
@stephenctw stephenctw force-pushed the feature/update-checkpoint-address branch from 9bd2f2a to 14cc4a8 Compare April 10, 2026 04:41
- LOG2_STRIDE);

pub const CHECKPOINT_ADDRESS: u64 = 0x7ffff000;
pub const CHECKPOINT_ADDRESS: u64 = 0xfe0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Instead of having this hardcoded, you could use CM_AR_SHADOW_REVERT_ROOT_HASH_START from the C API starting with emulator v0.20.0. Also we are calling this REVERT_ROOT_HASH on the emulator.

@stephenctw stephenctw changed the title feat: update CHECKPOINT_ADDRESS feat: update REVERT_ROOT_HASH_ADDRESS Apr 10, 2026
@stephenctw stephenctw force-pushed the feature/update-checkpoint-address branch from 14cc4a8 to ec077a2 Compare April 10, 2026 14:22
@stephenctw stephenctw requested a review from edubart April 10, 2026 15:05
@stephenctw stephenctw force-pushed the feature/update-checkpoint-address branch 2 times, most recently from 8a03100 to e88d495 Compare April 10, 2026 15:29
@stephenctw stephenctw force-pushed the feature/update-checkpoint-address branch from e88d495 to 3e07926 Compare April 10, 2026 15:37
@edubart
Copy link
Copy Markdown

edubart commented Apr 13, 2026

This is probably blocked by the PR bumping the emulator #241

@mpolitzer
Copy link
Copy Markdown

mpolitzer commented Apr 14, 2026

I updated the emulator bump PR but there is still one broken test, will investigate tomorrow. all tests are green.

ref. #241

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.

3 participants