← Back to index | Original Bugzilla link

Bug 17551 – VC build: bad mangling of real template argument

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P1
Component
dmd
Product
D
Version
D2
Platform
x86_64
OS
Windows
Creation time
2017-06-25T15:41:16Z
Last change time
2017-08-16T13:22:53Z
Keywords
pull
Assigned to
No Owner
Creator
Rainer Schuetze

Comments

Comment #0 by r.sagitario — 2017-06-25T15:41:16Z
When building dmd with the VC runtime, templates with real arguments are mangled wrong: module test; real foo(real x)() { return x; } pragma(msg, (foo!0.3).mangleof); // _D4test__T3fooVeex13333333333333334pN2ZQBdFNaNbNiNfZe Please note the additional 'x' before the constant as well as the small 'p' before the exponent (sould be 'P')
Comment #1 by r.sagitario — 2017-06-25T16:06:09Z
https://github.com/dlang/dmd/pull/6936
Comment #2 by github-bugzilla — 2017-06-25T18:58:17Z
Commits pushed to master at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6b0a15db30b37fc4556b65a18abf2fbd3c562f70 fix issue 17551 - VC build: bad mangling of real template argument fix ld_sprint not reflecting case of format character https://github.com/dlang/dmd/commit/16f4f4b9dcc5d585085944c2fdbc0a42c362e5d1 Merge pull request #6936 from rainers/issue17551 fix issue 17551 - VC build: bad mangling of real template argument
Comment #3 by github-bugzilla — 2017-08-07T13:16:59Z
Commits pushed to newCTFE at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6b0a15db30b37fc4556b65a18abf2fbd3c562f70 fix issue 17551 - VC build: bad mangling of real template argument https://github.com/dlang/dmd/commit/16f4f4b9dcc5d585085944c2fdbc0a42c362e5d1 Merge pull request #6936 from rainers/issue17551
Comment #4 by github-bugzilla — 2017-08-16T13:22:53Z
Commits pushed to stable at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6b0a15db30b37fc4556b65a18abf2fbd3c562f70 fix issue 17551 - VC build: bad mangling of real template argument https://github.com/dlang/dmd/commit/16f4f4b9dcc5d585085944c2fdbc0a42c362e5d1 Merge pull request #6936 from rainers/issue17551