[Issue 16574] [REG 2.072.0-b1] Unexplained errors about functions that overridde anything

[via Digitalmars-d-bugs]

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

--- Comment #14 from b2.temp at gmx.com ---
(In reply to Martin Nowak from comment #13)
> (In reply to b2.temp from comment #11)
> > $ if [ -d kheops ]; then
> > $     cd kheops
> > $     git pull
> > $     cd ..
> > $ else
> > $     git clone https://github.com/BBasile/kheops.git
> > $ fi
> > $ cd kheops/runnable
> > $ dub default_alignment.d 
> 
> Mmh, apparently you don't see the problem w/ your test case, so I'm saying
> this explicit.
> The moment you add a new commit, or any of the your project's dependencies
> gets updated, the bug might no longer be reproducible.

$ if ! [ -d kheops ]; then
$     git clone https://github.com/BBasile/kheops.git
$ fi
$ 
$ cd kheops
$ git checkout a00f300c4281b2d4aaf95e7fabde48d3dc3816e9
$ cd runnable
$ dub default_alignment.d

> I've already spend a lot of time trying to reproduce this bug w/ your ever

So do I. I've manually compiled DMD and phobos about a hundred of times to find
where default_alignment.d started not to work anymore. Also the (unfortunately)
wrong reduction didn't pop from the magician hat.

> moving example. So please make sure that bug reports use "permalinks"
> including the commit hash and dub.selections.json.
> 
> > I think that you did not see it because the unittest config does nothing
> > (it's a static library).
> 
> I ran dub build, if I recall correctly.

'dub build --build=unittest' does nothing, you should have get  "Target is a
library. Skipping execution. " I don't know what you are trying to tell me
here...anyway this is pointless because there's no unittest in this library.
The only unittest were only used to make static checks related to manual memory
management.

> (In reply to b2.temp from comment #12)
> > By the way Mr Nowak, I like to see how you're involved. But for this
> > particular regression, maybe Kenji Hara could get in...
> 
> Unlikely, he is hardly nowadays.

He's the author of the feature. I don't even understand why this is you that
handle all the dirty work with the many regressions of this pre-release.

> > If at a certain point he's not able to fix the new compiler feature:
> > 
> > https://dlang.org/changelog/2.072.0.html#deferred_alias
> > 
> > Then let's revert this feature until he has the time to work on it.
> 
> Well, your code involves a recursive template, and it's not clear whether
> it's actually possible to be compiled.

I have many others small applications like "default_alignment.d" that I run to
track possible breakage. Once again I don't understand what you are trying to
tell me. With DMD 2.071.2 they are all compiled, they run all.

> Lazy alias analysis is a huge improvement for compile times, 

Certainly.

> so we won't drop it light-hearted.

So don't drop it and make your release but keep in mind that there's an issue.
Soon or later it will show the tip of its nose again.

--