Bug 23065 – importC: __builtin_expect should use c_long
Status
RESOLVED
Resolution
FIXED
Severity
minor
Priority
P1
Component
druntime
Product
D
Version
D2
Platform
All
OS
All
Creation time
2022-04-27T23:51:59Z
Last change time
2022-04-28T08:02:58Z
Keywords
ImportC, pull
Assigned to
No Owner
Creator
duser
Comments
Comment #0 by duser — 2022-04-27T23:51:59Z
it's a C function documented as taking "long" but the implementation in druntime uses the D long type
Comment #1 by dlang-bot — 2022-04-27T23:59:18Z
@huglovefan created dlang/druntime pull request #3813 "fix Issue 23065: importC: __builtin_expect should use c_long" fixing this issue:
- fix Issue 23065: importC: __builtin_expect should use c_long
since it's a C function documented as taking "long"
https://github.com/dlang/druntime/pull/3813
Comment #2 by dlang-bot — 2022-04-28T08:02:58Z
dlang/druntime pull request #3813 "fix Issue 23065: importC: __builtin_expect should use c_long" was merged into master:
- a6ff2ae0a7be0cc3d77e8e2ac88ba5a30a2d6937 by human:
fix Issue 23065: importC: __builtin_expect should use c_long
since it's a C function documented as taking "long"
https://github.com/dlang/druntime/pull/3813