This is an archive of the discontinued LLVM Phabricator instance.

[Analysis] Remove unused class LegacyAARGetter
ClosedPublic

Authored by kazu on May 31 2023, 12:48 AM.

Details

Summary

The last use was removed by:

commit fa6ea7a419f37befbed04368bcb8af4c718facbb
Author: Arthur Eubanks <aeubanks@google.com>
Date:   Mon Mar 20 11:18:35 2023 -0700

Diff Detail

Event Timeline

kazu created this revision.May 31 2023, 12:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 31 2023, 12:48 AM
kazu requested review of this revision.May 31 2023, 12:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 31 2023, 12:48 AM
nikic added a subscriber: nikic.May 31 2023, 1:02 AM

I believe you can remove createLegacyPMAAResults() and createLegacyPMBasicAAResult() as well.

kazu updated this revision to Diff 527029.May 31 2023, 7:25 AM

Remove createLegacyPMAAResults and createLegacyPMBasicAAResult as well.

aeubanks accepted this revision.May 31 2023, 7:26 AM
This revision is now accepted and ready to land.May 31 2023, 7:26 AM
kazu added a comment.May 31 2023, 7:26 AM

I believe you can remove createLegacyPMAAResults() and createLegacyPMBasicAAResult() as well.

Thank you for pointing this out! I've updated the patch accordingly.

This revision was landed with ongoing or failed builds.May 31 2023, 7:30 AM
This revision was automatically updated to reflect the committed changes.