[PATCH] D73962: [DebugInfo] Error if unsupported address size detected in line table
Igor Kudrin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 4 07:18:43 PST 2020
ikudrin added inline comments.
================
Comment at: llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp:635
uint8_t ExtractorAddressSize = DebugLineData.getAddressSize();
- if (ExtractorAddressSize != Len - 1 && ExtractorAddressSize != 0)
+ // FIXME: Check to make sure Len <= sizeof(uint8_t).
+ uint8_t OpcodeAddressSize = Len - 1;
----------------
jhenderson wrote:
> I'm going to write a follow-up patch for this issue.
You probably meant `Len >= 1`, right?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73962/new/
https://reviews.llvm.org/D73962
More information about the llvm-commits
mailing list