Don't exclude functions in object.d from cross-module inlining

Resolves #1604. I had no issues compiling druntime and Phobos with
`-DD_FLAGS_RELEASE='-O3;-release;-enable-cross-module-inlining'`.
This commit is contained in:
Martin Kinkelin 2019-03-06 23:03:49 +01:00
parent 6c3e29eb0a
commit 40a82cd5e4

View file

@ -129,15 +129,6 @@ bool defineAsExternallyAvailable(FuncDeclaration &fdecl) {
return false;
}
// TODO: Fix inlining functions from object.d. Currently errors because of
// TypeInfo type-mismatch issue (TypeInfo classes get special treatment by the
// compiler). To start working on it: comment-out this check and druntime will
// fail to compile.
if (fdecl.getModule()->ident == Id::object) {
IF_LOG Logger::println("Inlining of object.d functions is disabled");
return false;
}
if (fdecl.semanticRun >= PASSsemantic3) {
// If semantic analysis has come this far, the function will be defined
// elsewhere and should not get the available_externally attribute from