Bug 19201 – Error: func called with argument types (ulong) matches both: __c_long and __c_ulong

Bootstrap builds with gdc are broken in master. Minimal version without dependencies. --- enum __c_long : long; enum __c_ulong : ulong; void a(ulong r){} void a(long r){} void b(__c_long r){} void b(__c_ulong r){} void main() { a(0L); b(0L); } --- Minimal version with dependencies. --- import core.stdc.stdint; void a(ulong r){} void a(long r){} void b(uint64_t r){} void b(int64_t r){} void main() { a(0L); b(0L); } --- Places in dmd that don't compile as a result: https://github.com/dlang/dmd/blob/05cdf2f55c9d6379e300382ab7af77837c6f4155/src/dmd/dcast.d#L377 https://github.com/dlang/dmd/blob/05cdf2f55c9d6379e300382ab7af77837c6f4155/src/dmd/dcast.d#L381 https://github.com/dlang/dmd/blob/05cdf2f55c9d6379e300382ab7af77837c6f4155/src/dmd/expressionsem.d#L9319 https://github.com/dlang/dmd/blob/05cdf2f55c9d6379e300382ab7af77837c6f4155/src/dmd/optimize.d#L938 The longdouble type has overrides for intXX_t and uintXX_t to be ABI compatible with C++.

Commits pushed to master at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/9772f49f99f0eaadad49f242f423c2cb1225031d fix Issue 19201 - func called with argument types (ulong) matches both: __c_long and __c_ulong https://github.com/dlang/dmd/commit/4b2c88529b3e9773e02991ecb53c27c1990881e5 Merge pull request #8632 from ibuclaw/issue19201 fix Issue 19201 - func called with argument types (ulong) matches both: __c_long and __c_ulong merged-on-behalf-of: Jacob Carlborg <[email protected]>

This should be broken. long is NOT __c_long, and treating the two as an exact match does not work. (For example, template matching now fails https://issues.dlang.org/show_bug.cgi?id=19499) The DMD code that doesn't compile should be fixed with a cast, not by breaking the type system.

Reopened as a regression, as this no longer compiles on OSX, but compiles on Linux (it should be compilable by all). --- import core.stdc.stdint; void func(uint64_t r){} void func(int64_t r){} void main() { func(0L); } --- Conversely, this no longer compiles on Linux, but compiles on OSX (it should be rejected by all). --- import core.stdc.stdint; void func(uint64_t r){} void func(int64_t r){} void func(ulong r){} void func(long r){} void main() { func(0L); } ---

(In reply to Iain Buclaw from comment #4) > Conversely, this no longer compiles on Linux, but compiles on OSX (it should > be rejected by all). Or compilable by all, either or, it is a problem that two platforms behave differently depending on what int64_t is an alias to.

(In reply to Iain Buclaw from comment #4) > Reopened as a regression, as this no longer compiles on OSX, but compiles on > Linux (it should be compilable by all). > --- > import core.stdc.stdint; > > void func(uint64_t r){} > void func(int64_t r){} > > void main() > { > func(0L); > } > --- > This does not compile on linux either > Conversely, this no longer compiles on Linux, but compiles on OSX (it should > be rejected by all). > --- > import core.stdc.stdint; > > void func(uint64_t r){} > void func(int64_t r){} > void func(ulong r){} > void func(long r){} > > void main() > { > func(0L); > } > --- This compiles on linux also. Maybe this should be closed?

(In reply to RazvanN from comment #6) > (In reply to Iain Buclaw from comment #4) > > Reopened as a regression, as this no longer compiles on OSX, but compiles on > > Linux (it should be compilable by all). > > --- > > import core.stdc.stdint; > > > > void func(uint64_t r){} > > void func(int64_t r){} > > > > void main() > > { > > func(0L); > > } > > --- > > > This does not compile on linux either Yes it does (v2.094.2-beta.1-559-g4522e0236) > > > Conversely, this no longer compiles on Linux, but compiles on OSX (it should > > be rejected by all). > > --- > > import core.stdc.stdint; > > > > void func(uint64_t r){} > > void func(int64_t r){} > > void func(ulong r){} > > void func(long r){} > > > > void main() > > { > > func(0L); > > } > > --- > > This compiles on linux also. > No it doesn't (v2.094.2-beta.1-559-g4522e0236)

THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/dmd/issues/19481 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB