[PATCH] D67097: [DWARF] Check for format mismatch between CU and Range List Table.

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 07:53:33 PDT 2019


ikudrin marked an inline comment as done.
ikudrin added inline comments.


================
Comment at: lib/DebugInfo/DWARF/DWARFUnit.cpp:311
+                        DwarfFormat Format) {
+  const uint64_t OffsetIn = Offset;
   // TODO: Support DWARF64
----------------
Orlando wrote:
> Is there a reason for OffsetIn to exist as opposed to just using Offset here?
`Offset` is changed in `Table.extractHeaderAndOffsets()` (line 323), but we have to show the original value in the error message (line 326).


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67097/new/

https://reviews.llvm.org/D67097





More information about the llvm-commits mailing list