Bug 17291 – [REG 2.074-b1] windows: invalid relocation entries

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
dmd
Product
D
Version
D2
Platform
x86_64
OS
Windows
Creation time
2017-04-02T07:28:00Z
Last change time
2017-08-07T13:15:36Z
Keywords
pull, wrong-code
Assigned to
nobody
Creator
r.sagitario

Comments

Comment #0 by r.sagitario — 2017-04-02T07:28:24Z
When compiling struct Struc(T) { static shared T* p; } void foo() { Struc!int.p = null; } with current stable or dmd 2.074-b1, using Agner Fog's objconv to disassemble the file reports: C:\tmp\d>objconv.exe -fasm ptr.obj Input file: ptr.obj, output file: ptr.asm Converting from OMF32 to Disassembly32 Checksums are zero Warning 1203: Unrecognized data in OMF subrecord This was introduced by https://github.com/dlang/dmd/pull/6534 and very likely is the cause for https://github.com/dlang/druntime/pull/1763 to fail.
Comment #1 by r.sagitario — 2017-04-02T08:26:53Z
Comment #2 by github-bugzilla — 2017-04-02T09:03:39Z
Commits pushed to stable at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/4b45720a9ec80d16af713164ccc56e649d3a83f6 fix issue 17291: relocations for pointer references might also need "back patches" https://github.com/dlang/dmd/commit/cd33499302cb27d0676c92035e3a6310adcb5c54 Merge pull request #6667 from rainers/issue17291 fix issue 17291: bad relocations for pointer references
Comment #3 by github-bugzilla — 2017-04-09T17:57:03Z
Commits pushed to master at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/4b45720a9ec80d16af713164ccc56e649d3a83f6 fix issue 17291: relocations for pointer references might also need "back patches" https://github.com/dlang/dmd/commit/cd33499302cb27d0676c92035e3a6310adcb5c54 Merge pull request #6667 from rainers/issue17291
Comment #4 by github-bugzilla — 2017-08-07T13:15:36Z
Commits pushed to newCTFE at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/4b45720a9ec80d16af713164ccc56e649d3a83f6 fix issue 17291: relocations for pointer references might also need "back patches" https://github.com/dlang/dmd/commit/cd33499302cb27d0676c92035e3a6310adcb5c54 Merge pull request #6667 from rainers/issue17291