Bug 22834 – runnable_cxx/stdint.d: Undefined reference to _Z15testCppI8Mangleahahah

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
druntime
Product
D
Version
D2
Platform
x86_64
OS
Solaris
Creation time
2022-03-01T19:51:35Z
Last change time
2022-03-03T00:53:29Z
Keywords
pull
Assigned to
No Owner
Creator
Iain Buclaw

Comments

Comment #0 by ibuclaw — 2022-03-01T19:51:35Z
E.g. int8_t on Solaris is char because char is signed by default. Caused by https://github.com/dlang/druntime/pull/3400
Comment #1 by dlang-bot — 2022-03-01T21:07:48Z
@ibuclaw updated dlang/druntime pull request #3769 "fix Issue 22834 - core.stdc.stdint aliases are not platform independent" fixing this issue: - fix Issue 22834 - core.stdc.stdint aliases are not platform independent https://github.com/dlang/druntime/pull/3769
Comment #2 by Ajieskola — 2022-03-01T21:32:56Z
Are you sure this is really a regression? If Solaris mangles int8_t as C++ char, that is a good reason to make it and uint8_t platform-dependant again. But if so, it should have been broken before pull request 3400 already. There was no special casing for Solaris, and Posix int8_t was defined as ubyte just like now.
Comment #3 by dlang-bot — 2022-03-03T00:53:29Z
dlang/druntime pull request #3769 "fix Issue 22834 - core.stdc.stdint aliases are not platform independent" was merged into master: - 02bb15a3019e90f995abda3a6d5b60cffe8aefc4 by Iain Buclaw: fix Issue 22834 - core.stdc.stdint aliases are not platform independent https://github.com/dlang/druntime/pull/3769