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 :).

struct S
{
    S* s;
    S** ss;
    S*[4] s4;
}

void main()
{
    S s;
}