Treating DST constructor as a function aborts compiler

[Stebalien]

Aborts (no backtrace):

struct Test([i32]);
fn main() {
  let x = Test;
}

With:

rustc: /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/llvm/lib/IR/Instructions.cpp:995: void llvm::StoreInst::AssertOK(): Assertion `getOperand(0)->getType() == cast<PointerType>(getOperand(1)->getType())->getElementType() && "Ptr must be a pointer to Val type!"' failed.

Forked from: #30240.

[Stebalien]

According to @Aatch,

The last error is the most complex as it's a weird case of struct Test([i32]) being a perfectly valid type, but fn([i32]) -> Test not being a valid function. We probably need to check whether you're forcing the creation of an invalid function and error somewhere earlier. That's where the error comes from, btw, the creation of the fn Test(_0: [i32]) -> Test { ... } function body.

[Aatch]

The minimum reproduction is this:

struct Test([i32]);
fn main() {
  let x = Test;
}

Just to show that it's not got anything to do with taking a reference.

[Stebalien]

@Aatch, updated.

[steveklabnik]

This no longer ICEs!

[eddyb]

@steveklabnik Not so fast:

struct Test([i32]);
fn main() {
  let x: fn(_) -> Test = Test;
}

Reification is now lazy, Test itself is a zero-sized function item.

[nikomatsakis]

cc me

[Stebalien]

This appears to have been fixed.