This is an archive of the discontinued LLVM Phabricator instance.

[DebugInfo][test] Adjust line table unit length to account for contents
ClosedPublic

Authored by jhenderson on Jan 31 2020, 2:55 AM.

Details

Summary

Previously, if a debug line Prologue was created via createBasicPrologue, its TotalLength field did not account for any contents in the table itself. This change fixes this issue.

Diff Detail

Event Timeline

jhenderson created this revision.Jan 31 2020, 2:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 31 2020, 2:55 AM
This revision is now accepted and ready to land.Jan 31 2020, 8:02 AM
This revision was automatically updated to reflect the committed changes.
jhenderson marked an inline comment as done.Feb 3 2020, 8:30 AM
jhenderson added inline comments.
llvm/unittests/DebugInfo/DWARF/DwarfGenerator.cpp
178

Oops, this is incorrect. Contents is not a vector of bytes of data, but rather of ValueAndLength entries, which potentially represent different numbers of bytes. I'll post a fix.

jhenderson marked an inline comment as done.Feb 3 2020, 8:55 AM
jhenderson added inline comments.
llvm/unittests/DebugInfo/DWARF/DwarfGenerator.cpp
178

I'll post a fix.

See D73901.