This is an archive of the discontinued LLVM Phabricator instance.

[LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC.
ClosedPublic

Authored by ikudrin on Oct 15 2020, 7:58 AM.

Details

Summary

This is a follow-up for D62935 where the corresponding API methods were removed.

Diff Detail

Event Timeline

ikudrin created this revision.Oct 15 2020, 7:58 AM
ikudrin requested review of this revision.Oct 15 2020, 7:58 AM
steven_wu accepted this revision.Oct 16 2020, 8:57 AM
This revision is now accepted and ready to land.Oct 16 2020, 8:57 AM
dexonsmith accepted this revision.Oct 19 2020, 5:47 PM
dblaikie added a subscriber: dblaikie.EditedOct 27 2020, 1:46 PM

@dexonsmith / @steven_wu - just a quick note that phab approvals without any text don't generate email/show up on the mailing list (leading to it looking like patches are committed without review approval) - if you can include some text in phab approvals it'd be helpful
maybe we'll be able to get this fixed in Phab at some point

@dexonsmith / @steven_wu - just a quick note that phab approvals without any text don't generate email/show up on the mailing list (leading to it looking like patches are committed without review approval) - if you can include some text in phab approvals it'd be helpful
maybe we'll be able to get this fixed in Phab at some point

Thanks, noted! (I think I knew that before but forgot along the way...)