D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 23065 - importC: __builtin_expect should use c_long
Summary: importC: __builtin_expect should use c_long
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: druntime (show other issues)
Version: D2
Hardware: All All
: P1 minor
Assignee: No Owner
URL:
Keywords: ImportC, pull
Depends on:
Blocks:
 
Reported: 2022-04-27 23:51 UTC by duser
Modified: 2022-04-28 08:02 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description duser 2022-04-27 23:51:59 UTC 
it's a C function documented as taking "long" but the implementation in druntime uses the D long type
Comment 1 Dlang Bot 2022-04-27 23:59:18 UTC 
@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 Dlang Bot 2022-04-28 08:02:58 UTC 
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