This is an archive of the discontinued LLVM Phabricator instance.

Don't link ObjCARCOpts twice. Fixes PR22543
ClosedPublic

Authored by loladiro on Jun 11 2015, 12:49 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

loladiro updated this revision to Diff 27537.Jun 11 2015, 12:49 PM
loladiro retitled this revision from to Don't link ObjCARCOpts twice. Fixes PR22543.
loladiro updated this object.
loladiro edited the test plan for this revision. (Show Details)
loladiro added a reviewer: sylvestre.ledru.
loladiro set the repository for this revision to rL LLVM.
loladiro added a subscriber: Unknown Object (MLST).
loladiro updated this revision to Diff 27541.Jun 11 2015, 1:48 PM

While I'm at it, fix a related problem in lldb-server.

Bump. This is a minor change without which LLDB is totally unusable when built with Makefiles. Should I generally just commit these kinds of things directly? There's also D5431, which I'd like to get in.

labath accepted this revision.Jun 18 2015, 1:42 PM
labath added a reviewer: labath.
labath added a subscriber: labath.

We don't use the makefile build, but it looks pretty safe. I say just go ahead...

This revision is now accepted and ready to land.Jun 18 2015, 1:42 PM
This revision was automatically updated to reflect the committed changes.