← Back to index | Original Bugzilla link

Bug 11674 – core.stdc.fenv.fenv_t declaration not architecture aware

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
druntime
Product
D
Version
D2
Platform
All
OS
Linux
Creation time
2013-12-03T17:00:00Z
Last change time
2015-06-09T05:15:22Z
Keywords
pull
Assigned to
nobody
Creator
code

Comments

Comment #0 by code — 2013-12-03T17:00:17Z
The declaration of fenv_t is architecture dependent but druntime unconditionally uses the X86 one.
Comment #1 by code — 2013-12-05T03:26:54Z
https://github.com/D-Programming-Language/druntime/pull/683
Comment #2 by github-bugzilla — 2013-12-05T12:52:48Z
Commits pushed to master at https://github.com/D-Programming-Language/druntime https://github.com/D-Programming-Language/druntime/commit/fc9a485c96305d38171fcd30c820437b4ca19831 fix Issue 11674 - fenv_t declaration not architecture aware - add declarations for X86 and X86_64 - also fix incorrect type for fexcept_t https://github.com/D-Programming-Language/druntime/commit/1ebc320ed53c01b4c521087c287d69e902a8712a Merge pull request #683 from dawgfoto/fix11674 fix Issue 11674 - fenv_t declaration not architecture aware
Comment #3 by github-bugzilla — 2013-12-19T05:48:58Z
Commits pushed to 2.065 at https://github.com/D-Programming-Language/druntime https://github.com/D-Programming-Language/druntime/commit/fc9a485c96305d38171fcd30c820437b4ca19831 fix Issue 11674 - fenv_t declaration not architecture aware https://github.com/D-Programming-Language/druntime/commit/1ebc320ed53c01b4c521087c287d69e902a8712a Merge pull request #683 from dawgfoto/fix11674
Comment #4 by dbugz — 2015-04-26T11:41:51Z
It appears this has since been fixed: https://github.com/D-Programming-Language/druntime/blob/master/src/core/stdc/fenv.d#L30