Skip to content

math.unsigned: fix rotate_left() for uint256, add test - #24872

Merged
spytheman merged 1 commit into
vlang:masterfrom
tankf33der:rotate_left_256
Jul 10, 2025
Merged

spytheman merged 1 commit into
vlang:masterfrom
tankf33der:rotate_left_256

Conversation

@tankf33der

@tankf33der tankf33der commented Jul 9, 2025 •

Copy link
Copy Markdown
Contributor

Lets fix rotate_left() too.
Without this patch, the current rotate_left() cannot pass this test.

Passed 1B loop against loop gmplib.

@huly-for-github

Copy link
Copy Markdown

Connected to Hulyยฎ: V_0.6-23303

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Excellent work.

@spytheman

Copy link
Copy Markdown
Contributor

Thank you for taking the time to go over the methods in math.unsigned methodically, and verify and fix them ๐Ÿ™‡๐Ÿปโ€โ™‚๏ธ @tankf33der .

@spytheman
spytheman merged commit 03e72a4 into vlang:master Jul 10, 2025
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.

2 participants