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

--- Comment #14 from [email protected] ---
(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.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.

--

Reply via email to