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

[REG2.064] Issue 15210 - [ICE] (glue.c at 1489) with tuples and AAs #5209

Closed
wants to merge 1 commit into from

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Oct 20, 2015

@@ -3787,6 +3787,7 @@ extern (C++) Expression addInvariant(Loc loc, Scope* sc, AggregateDeclaration ad
se = se.semantic(sc);
se.type = Type.tchar.arrayOf();
e = new AssertExp(loc, v, se);
e.type = Type.tvoid;
Copy link
Member

Choose a reason for hiding this comment

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

When is an AssertExp ever not void?

Copy link
Member

Choose a reason for hiding this comment

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

When it still needs semantic analysis run on it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, I fixed to call semantic on the AssertExp.

@9rnsr
Copy link
Contributor Author

9rnsr commented Oct 21, 2015

Sorry, this is based on master, it's incorrect. Replace with #5216.

@9rnsr 9rnsr closed this Oct 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants