Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 10086 & 10857 - Adjust instantiated template process order, and fix ICE in glue.c #2485

Merged
merged 5 commits into from Aug 27, 2013

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Aug 20, 2013

Issue 10086 - ICE(glue.c) or wrong code on passing variable as template value parameter
Issue 10857 - ICE(glue.c, bugzilla 2962?) or compiles, depending on the files order

@ghost
Copy link

ghost commented Aug 20, 2013

@9rnsr Please look at http://d.puremagic.com/issues/show_bug.cgi?id=10086 - it is also related to issue 2962

@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 20, 2013

Thanks for the mention, @mxfm .
I found that bug 10086, 10857, and bug 4418 had essentially same root cause. Updated summary.

{ VarDeclaration *v = (*parameters)[i];
if (v->csym)
{
error("compiler error, parameter '%s', bugzilla 2962?", v->toChars());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that this is going away!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I'm happy to do it!

WalterBright added a commit that referenced this pull request Aug 27, 2013
Issue 10086 & 10857 - Adjust instantiated template process order, and fix ICE in glue.c
@WalterBright WalterBright merged commit 2747184 into dlang:master Aug 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants