Skip to content

RNG-191: Dynamically call Math multiply high methods#192

Merged
aherbert merged 1 commit into
apache:masterfrom
aherbert:unsigned-multiply
Mar 8, 2026
Merged

RNG-191: Dynamically call Math multiply high methods#192
aherbert merged 1 commit into
apache:masterfrom
aherbert:unsigned-multiply

Conversation

@aherbert
Copy link
Copy Markdown
Contributor

@aherbert aherbert commented Mar 3, 2026

Adds unsignedMultiplyHigh methods potentially using native 128-bit multiplication to the Philox4x64 generator.

Adds unsignedMultiplyHigh methods potentially using native 128-bit
multiplication to the Philox4x64 generator.
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.56%. Comparing base (1e43bd4) to head (71f49b5).
⚠️ Report is 205 commits behind head on master.

Files with missing lines Patch % Lines
...pache/commons/rng/core/source64/PhiloxSupport.java 64.28% 10 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #192      +/-   ##
============================================
- Coverage     99.69%   99.56%   -0.14%     
+ Complexity     1495      813     -682     
============================================
  Files           152      158       +6     
  Lines          5996     6436     +440     
  Branches        564      593      +29     
============================================
+ Hits           5978     6408     +430     
- Misses           12       22      +10     
  Partials          6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aherbert aherbert merged commit d73cbc4 into apache:master Mar 8, 2026
7 checks passed
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