[Issue 12958] core.checkedint.mulu is broken

via Digitalmars-d-bugs digitalmars-d-bugs at puremagic.com
Mon Jul 14 15:35:42 PDT 2014


https://issues.dlang.org/show_bug.cgi?id=12958

--- Comment #11 from github-bugzilla at puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/3787f9b79d807631af37a986a8a164a192413a1b
Fix issue 12958 - core.checkedint.mulu is broken.

This implements the (uint, uint) case using ulong arithmetic
and the (ulong, ulong) case using trial division. See Bugzilla
for an ulong implementation by David Bregman that does not
require a division. As the functions are expected to be
implemented as compiler intrinsics to make use of
architecture-specific primitives anyway, the simpler version
has been chosen to reduce the potential for bugs, even though
it is slower.

https://github.com/D-Programming-Language/druntime/commit/7d2a2ef77fd07796ef1e96ec1ef5fac9016f511d
Merge pull request #890 from klickverbot/fix-checkedint-umul

Fix issue 12958 - core.checkedint.mulu is broken.

--


More information about the Digitalmars-d-bugs mailing list