← Back to index | Original Bugzilla link

Bug 20197 – Make std.math.isIdentical work in CTFE

Status: NEW
Severity: enhancement
Priority: P4
Component: phobos
Product: D
Version: D2
Platform: All
OS: All
Creation time: 2019-09-08T16:13:35Z
Last change time: 2024-12-07T09:36:06Z
Keywords: pull
Assigned to: No Owner
Creator: Nathan S.
Blocks: 17007
See also: https://issues.dlang.org/show_bug.cgi?id=17007

Comments

Comment #0 by n8sh.secondary — 2019-09-08T16:13:35Z

Making std.math.isIdentical work for real NaN when real is more than 64 bits requires either changing `is` for floats to be a binary comparison (like https://github.com/dlang/dmd/pull/7568) or adding support for CTFE reinterpretation of reals as byte arrays. This is a subissue of Issue #17007.

Comment #1 by n8sh.secondary — 2020-06-06T00:16:16Z

Actually it doesn't require this since in CTFE there is no way to construct a real nan with a payload that would be lost when casting to double.

Comment #2 by dlang-bot — 2020-06-06T01:32:49Z

@n8sh created dlang/phobos pull request #7515 "Make std.math.isIdentical work in CTFE" fixing this issue: - Fix Issue 20197 - Make std.math.isIdentical work in CTFE https://github.com/dlang/phobos/pull/7515

Comment #3 by robert.schadek — 2024-12-01T16:35:32Z

THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10386 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB

Comment #4 by dlang-bot — 2024-12-07T09:36:06Z

@runiixx updated dlang/phobos pull request #10576 "Make std.math.isIdentical work in CTFE with coverage tests" fixing this issue: - Fix Bugzilla Issue 20197 Signed-off-by: runiixx <[email protected]> https://github.com/dlang/phobos/pull/10576