Issue 20197 - Make std.math.isIdentical work in CTFE
Summary: Make std.math.isIdentical work in CTFE
Status: NEW
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: All All
: P4 enhancement
Assignee: No Owner
URL:
Keywords: pull
Depends on:
Blocks: 17007
  Show dependency tree / graph
 
Reported: 2019-09-08 16:13 UTC by Nathan S.
Modified: 2024-12-07 09:36 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Nathan S. 2019-09-08 16:13:35 UTC 
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 Nathan S. 2020-06-06 00:16:16 UTC 
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 Dlang Bot 2020-06-06 01:32:49 UTC 
@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 dlangBugzillaToGithub 2024-12-01 16:35:32 UTC 
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 Dlang Bot 2024-12-07 09:36:06 UTC 
@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 <murgua03@gmail.com>

https://github.com/dlang/phobos/pull/10576