[PATCH] D32713: [DWARFv5] Parse new line-table header format.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 14:33:35 PDT 2017


dblaikie accepted this revision.
dblaikie added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/DebugInfo/DWARF/DWARFDebugLine.cpp:143
+  // Get the directory entry description.
+  SmallVector<ContentDescriptor, 4> DirDescriptors =
+    parseV5EntryFormat(DebugLineData, OffsetPtr, EndPrologueOffset);
----------------
Use the ContentDescriptors typedef here? (I'd probably skip the typedef/not use it at all - but if you're going to use it, probably makes sense to use it here?)


https://reviews.llvm.org/D32713





More information about the llvm-commits mailing list