Fix memory leak

[valeriyvan]

No description provided.

[artemcm]

Ah, thank you very much @valeriyvan.
What do you think about hoisting the code block:

if (localSubInstanceMap)
    delete subInstanceMap;

To right after we define subInstanceMap and in a SWIFT_DEFER block?

That way adding new returns to this method won't require the developer to remember to do this.

[valeriyvan]

Ah, thank you very much @valeriyvan. What do you think about hoisting the code block:

if (localSubInstanceMap)
    delete subInstanceMap;

To right after we define subInstanceMap and in a SWIFT_DEFER block?

That way adding new returns to this method won't require the developer to remember to do this.

done

[artemcm]

@swift-ci test and merge

[artemcm]

@swift-ci test and merge

[artemcm]

@swift-ci test

[valeriyvan]

ping

[AnthonyLatsis]

@valeriyvan Could you squash the commits and add a "DependencyScan:" or similar tag at the start of the message?

[valeriyvan]

@valeriyvan Could you squash the commits and add a "DependencyScan:" or similar tag at the start of the message?

done

[AnthonyLatsis]

@swift-ci please smoke test and merge

[AnthonyLatsis]

Thanks!