This is an archive of the discontinued LLVM Phabricator instance.

[hmaptool] Fix dumping
ClosedPublic

Authored by smeenai on Jan 23 2022, 12:54 PM.

Details

Summary

It was complaining about too many values to unpack, since our struct
unpack format string specified six members, but we only had five
variables to unpack to. The sixth value is the max value length, but
it's not used in dumping, so we can ignore it.

Diff Detail

Event Timeline

smeenai requested review of this revision.Jan 23 2022, 12:54 PM
smeenai created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 23 2022, 12:54 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
bruno accepted this revision.Mar 3 2022, 3:05 PM

LGTM

This revision is now accepted and ready to land.Mar 3 2022, 3:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 3 2022, 3:05 PM
This revision was automatically updated to reflect the committed changes.