Conversation
…RTERS The original AVX512 implementation used incorrect 4-argument macros that only shuffled within single registers. For proper row/column processing, AVX512 requires inter-register shuffles using: - SWAP_HALVES for row processing (BLAKE2_ROUND_1_AVX512) - SWAP_QUARTERS/UNSWAP_QUARTERS for column processing (BLAKE2_ROUND_2_AVX512) This matches the PHC reference implementation's approach and ensures correct hash output with -O3 -march=native optimizations. Part of the fix for commit c0e59cf 'Rinhash: add native CPU optimizations' which introduced buggy SIMD code for AVX2, SSSE3, and AVX512.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release v1.0.2
Overview
This release introduces significant performance improvements, enhanced CPU optimizations, and comprehensive testnet/regtest network parameter updates.
Key Features
🚀 Performance Improvements
⚡ Rinhash CPU Optimizations
🌐 Network Updates
Technical Changes
Modified Files (27 files, +2164/-113 lines):
src/crypto/argon2/with multiple SIMD variantssrc/chainparams.cppandsrc/chainparamsseeds.hsrc/qt/coincontroldialog.cppandsrc/qt/modaloverlay.cppconfigure.acandsrc/Makefile.amCommits
c0e59cfRinhash: add native CPU optimizations3bb8c83Speed up Qt sync and payment dialogsde08fd4Chainparams: new dns+ip seeds, complete new testnet+regtest params incl. genesisbc6f40bsequential blockfile read0757187TestNet chainparams + version update 1.0.2Version
Bumped to v1.0.2
Testing