[Lldb-commits] [PATCH] D66789: Remove DWARFExpression::LocationListSize
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Aug 27 01:35:11 PDT 2019
labath created this revision.
labath added reviewers: JDevlieghere, clayborg.
Herald added subscribers: kristof.beyls, javed.absar, aprantl.
The only reason for this function's existance is so that we could pass
the correct size into the DWARFExpression constructor. However, there is
no harm in passing the entire data extractor into the DWARFExpression,
since the same code is performing the size determination as well as the
subsequent parse. So, if we get malformed input or there's a bug in the
parser, we'd compute the wrong size anyway.
Additionally, reducing the number of entry points into the location list
parsing machinery makes it easier to switch the llvm debug_loc(lists)
While inside, I added a couple of tests for invalid location list
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 8438 bytes
Desc: not available
More information about the lldb-commits