This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Fix --max-funcs=<N> option
ClosedPublic

Authored by maksfb on Jun 12 2023, 1:48 PM.

Details

Summary

Fix off-by-one error while handling of the --max-funcs=<N> option.
We used to process N+1 functions when N was requested.

Diff Detail

Event Timeline

maksfb created this revision.Jun 12 2023, 1:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 1:48 PM
Herald added a subscriber: treapster. · View Herald Transcript
maksfb requested review of this revision.Jun 12 2023, 1:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 1:48 PM
Amir accepted this revision.Jun 12 2023, 4:29 PM
This revision is now accepted and ready to land.Jun 12 2023, 4:29 PM
This revision was automatically updated to reflect the committed changes.