https://issues.dlang.org/show_bug.cgi?id=2789

Seb <greensunny12@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #17 from Seb <greensunny12@gmail.com> ---
Temporarily reopened due to regressions reported.
See also https://github.com/dlang/dmd/pull/7969,
https://github.com/dlang/dmd/pull/7577

--

https://issues.dlang.org/show_bug.cgi?id=2789

--- Comment #18 from Witold Baryluk <witold.baryluk+d@gmail.com> ---
Interesting, gdc 9.2 does report duplicate:

<source>:16:6: error: example.B.m at <source>:16:6 conflicts with example.B.m at <source>:12:6

   16 |  int m() {



The issue is still present in dmd nightly, dmd 2.089, ldc 1.20, ldc trunk.

Link with an example: https://godbolt.org/z/xE6fK7

--

https://issues.dlang.org/show_bug.cgi?id=2789

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ibuclaw@gdcproject.org

--- Comment #19 from Iain Buclaw <ibuclaw@gdcproject.org> ---
(In reply to Witold Baryluk from comment #18)
> Interesting, gdc 9.2 does report duplicate:
> 
> <source>:16:6: error: example.B.m at <source>:16:6 conflicts with example.B.m at <source>:12:6
> 
>    16 |  int m() {
> 
> 
> 
> The issue is still present in dmd nightly, dmd 2.089, ldc 1.20, ldc trunk.
> 
> Link with an example: https://godbolt.org/z/xE6fK7

This is checked in the codegen pass.  Would be better if the front-end did it for us.

--

https://issues.dlang.org/show_bug.cgi?id=2789

--- Comment #20 from Witold Baryluk <witold.baryluk+d@gmail.com> ---
This appears to have been fixed at some point, as I cannot reproduce it anymore:

$ dmd t.d
t.d(6): Error: function `t.A.m()` conflicts with previous declaration at t.d(2)
t.d(16): Error: function `t.B.m()` conflicts with previous declaration at
t.d(12)
$ ldc t.d
ldc2               ldc-build-runtime  ldc-profdata       ldc-prune-cache
$ ldc2 t.d
t.d(6): Error: function `t.A.m()` conflicts with previous declaration at t.d(2)
t.d(16): Error: function `t.B.m()` conflicts with previous declaration at
t.d(12)
$ gdc t.d
t.d:6:11: error: function ‘t.A.m()’ conflicts with previous declaration at
t.d:2:9
    6 |     float m() {
      |           ^
t.d:16:9: error: function ‘t.B.m()’ conflicts with previous declaration at
t.d:12:9
   16 |     int m() {
      |         ^


Same with global functions.


DMD64 D Compiler v2.105.2

LDC - the LLVM D compiler (1.30.0):
  based on DMD v2.100.1 and LLVM 14.0.6

gdc (Debian 13.2.0-5) 13.2.0

--

https://issues.dlang.org/show_bug.cgi?id=2789

--- Comment #21 from Witold Baryluk <witold.baryluk+d@gmail.com> ---
dmd 2.094.2 - incorrectly compiles successfully

dmd 2.095.0 - correctly shows error and aborts

--

https://issues.dlang.org/show_bug.cgi?id=2789

--- Comment #22 from Witold Baryluk <witold.baryluk+d@gmail.com> ---
Ah yes of course: https://dlang.org/changelog/2.095.0.html#duplicate-implementations-deprecation

Unfortunately this changelog does not link bugzilla or pull requests with fixes.

--

https://issues.dlang.org/show_bug.cgi?id=2789

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
                 CC|                            |b2.temp@gmx.com
         Resolution|---                         |FIXED

--- Comment #23 from Basile-z <b2.temp@gmx.com> ---
fixed by https://github.com/dlang/dmd/pull/8429

--