Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-Authored-By: Günther Foidl <gue@korporal.at>
  • Loading branch information
ycrumeyrolle and gfoidl committed Feb 18, 2020
1 parent 8b29e0a commit 2e74e9b
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ private static uint FastMod3(uint value)

ulong lowbits = (ulong.MaxValue / 3 + 1) * value;

// 64bit * 64bit => 128bit isn't currently supported by Math https://github.com/dotnet/corefx/issues/41822
// 64bit * 64bit => 128bit isn't currently supported by Math https://github.com/dotnet/runtime/issues/31184
// otherwise we'd want this to be (uint)Math.BigMul(lowbits, divisor, out _)
uint highbits = (uint)((((lowbits >> 32) + 1) * 3) >> 32);

Expand Down

0 comments on commit 2e74e9b

Please sign in to comment.