Page MenuHomePhabricator

Initialize ObjC runtime at the right location.
ClosedPublic

Authored by sas on Mar 23 2015, 11:15 AM.

Details

Summary

Saw this while reading some code in DynamicLoader classes. Looks like this has
been a FIXME since 2011 at least.

Diff Detail

Repository
rL LLVM

Event Timeline

sas updated this revision to Diff 22491.Mar 23 2015, 11:15 AM
sas retitled this revision from to Initialize ObjC runtime at the right location..
sas updated this object.
sas edited the test plan for this revision. (Show Details)
sas added a reviewer: clayborg.
sas added a subscriber: Unknown Object (MLST).
clayborg accepted this revision.Mar 23 2015, 11:35 AM
clayborg edited edge metadata.

lgtm

This revision is now accepted and ready to land.Mar 23 2015, 11:35 AM
This revision was automatically updated to reflect the committed changes.

Why are we putting logic for a specific language runtime in generic code?
This doesn't seem right to me.