Runtime part of issue 10023.

[jacob-carlborg]

Depends on dlang/dmd#2271.

[MartinNowak]

newline

[jacob-carlborg]

Right.

[MartinNowak]

LGTM

[jacob-carlborg]

I forgot, I had a couple of questions;

• Should we add new functionality like this to ModuleInfo.Old?
• Should I replace ModuleInfo.Old.reserved with the new field?
• Is it ok I add the new field before the existing name field in the new layout?

[MartinNowak]

I think we can safely dismiss the old ModuleInfo stuff, should be really old by now.

Is it ok I add the new field before the existing name field in the new layout?

That should be OK, if you used an older compiler the missing field won't change the layout.

[alexrp]

Speaking of, can't we kill off the old minfo stuff by now?

[jacob-carlborg]

Currently the field of the Old struct is used to get the size of the field to calculate offsets.

[quickfur]

ping

[jacob-carlborg]

@quickfur This is on hold because of the compiler part. In that pull request a long discussion started about that we should support some kind of user defined RTInfo before implementing this.

[andralex]

Status of this?

[jacob-carlborg]

Status of this?

See: dlang/dmd#2271 (comment).

[MartinNowak]

I'll close this, see dlang/dmd#2271 (comment).