Skip to content

[PoW][RandomX] Cherry picked fixes from offical randomx repo#891

Merged
codeofalltrades merged 6 commits intoVeil-Project:masterfrom
codeofalltrades:pow-changes
Jan 21, 2021
Merged

[PoW][RandomX] Cherry picked fixes from offical randomx repo#891
codeofalltrades merged 6 commits intoVeil-Project:masterfrom
codeofalltrades:pow-changes

Conversation

@codeofalltrades
Copy link
Collaborator

Problem

Keep randomx code current

Solution

Cherry picked fixes from offical randomx repo
Fix alignment for ARMv8 code - tevador/RandomX@9a5d504
Fix inconsistent class/struct usage - tevador/RandomX@57545d1
Combined hash and fill AES loop - tevador/RandomX@219c02e
Hide tempHash from the public API - tevador/RandomX@3910d49
Replace hardcoded literal with its appropriate symbol - tevador/RandomX@a7733de
Fix symbol collisions with blake2b - tevador/RandomX@7567cef

Unit Testing Results

Start the wallet
Start mining randomx
Verify you can spend mined coins

Copy link
Collaborator

@CaveSpectre11 CaveSpectre11 left a comment

Choose a reason for hiding this comment

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

ACK f033b57

@codeofalltrades codeofalltrades merged commit fbd5fd4 into Veil-Project:master Jan 21, 2021
@codeofalltrades codeofalltrades deleted the pow-changes branch January 21, 2021 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Algo: RandomX Code Review: Passed Component: Miner Both PoW and PoS block creation Tag: PoW Related to Proof of Work consensus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants