This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objcopy] Remove unneeded checks. NFC
ClosedPublic

Authored by evgeny777 on Mar 7 2019, 1:58 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

evgeny777 created this revision.Mar 7 2019, 1:58 AM
grimar accepted this revision.Mar 7 2019, 2:02 AM

LGTM

This revision is now accepted and ready to land.Mar 7 2019, 2:02 AM
jhenderson accepted this revision.Mar 7 2019, 2:51 AM

LGTM, personally, but I know @jakehehrlich had some thoughts on this, so I'd like him to give a thumbs up on it before this lands. See for example comments on D49979.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 12 2019, 5:43 AM