Skip to content

Commit

Permalink
[hmaptool] Fix dumping
Browse files Browse the repository at this point in the history
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.

Reviewed By: bruno

Differential Revision: https://reviews.llvm.org/D118004
  • Loading branch information
smeenai committed Mar 3, 2022
1 parent bc1574b commit 0c1d330
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion clang/utils/hmaptool/hmaptool
Expand Up @@ -44,7 +44,7 @@ class HeaderMap(object):
path,))

(version, reserved, strtable_offset, num_entries,
num_buckets) = struct.unpack(header_fmt, data)
num_buckets, _) = struct.unpack(header_fmt, data)

if version != 1:
raise SystemExit("error: %s: unknown headermap version: %r" % (
Expand Down

0 comments on commit 0c1d330

Please sign in to comment.