Bug 5026 – ICE(expression.c) Incomplete mixin expression + char[] to char assignment

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
Other
OS
Windows
Creation time
2010-10-09T11:10:00Z
Last change time
2015-06-09T05:12:01Z
Keywords
ice-on-invalid-code, patch
Assigned to
nobody
Creator
strtr

Comments

Comment #0 by strtr — 2010-10-09T11:10:11Z
Assertion failure: '0' on line 1342 in file 'expression.c' void foo() { char[0] code; code[0] = ""; } struct Bar { int i = mixin(foo()); } void main() {}
Comment #1 by clugdbug — 2010-10-18T13:27:37Z
There are two independent issues here. The superficial diagnostic bug is that mixin() shouldn't accept a void function at all. expression.c, line 5569. Actually the second error message here is optional, since there should have already been a "cannot be evaluated at compile time" error message. Expression *CompileExp::semantic(Scope *sc) { #if LOGSEMANTIC printf("CompileExp::semantic('%s')\n", toChars()); #endif UnaExp::semantic(sc); e1 = resolveProperties(sc, e1); + if (!e1->type->isString()) { + error("argument to mixin must be a string, not (%s)\n", e1->toChars()); + return new ErrorExp(); + } e1 = e1->optimize(WANTvalue | WANTinterpret); if (e1->op != TOKstring) - { error("argument to mixin must be a string, not (%s)\n", e1->toChars()); + { error("argument to mixin must be evaluated at compile time"); return new ErrorExp(); } The more important bug, which causes the ICE, is that declaration.c line 1212 is gagging the results. Then, interpret.c runs semantic3 on the function while results are gagged. The errors get suppressed, so garbage gets passed to the back end. func.c, semantic3(), line 1592. sc2->callSuper = 0; sc2->pop(); } + if (global.gag && global.errors) + semanticRun = PASSsemanticdone; // Ensure errors get reported again + else semanticRun = PASSsemantic3done; } void FuncDeclaration::toCBuffer(OutBuffer *buf, HdrGenState *hgs)
Comment #2 by clugdbug — 2010-10-18T13:29:49Z
And a couple of test cases. string bug5026() { char[0] code; code[0] = "0"; return "0"; } template compiles(int T) { bool compiles = true; } // ICE(glue.c) static assert(!is(typeof(compiles!(mixin(bug5026()))))); // ICE(expression.c) static assert(!is(typeof(mixin(bug5026())))); // This example is an accepts-valid bug string bug5026c() { char[0] code; code[0] = 7; return "0"; } static assert(!is(typeof(mixin(bug5026c()))));
Comment #3 by bugzilla — 2010-10-19T22:52:31Z