Clean up some suspicious code. There's no guarantee a constant of integer type is a ConstantInt, so it's not safe to just cast<> these to ConstantInt. (It could be a constant expression, for example) The code is shorter this way too :).

module bug8;

void main()
{
    s = newS();
}

S* s;

struct S
{
    int i;
}

S* newS()
{
    auto tmp = new S;
    tmp.i = 4;
    return tmp;
}