[Lldb-commits] [PATCH] D62302: DWARF: Fix address range support in mixed 4+5 scenario

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In D62302#1513824 <https://reviews.llvm.org/D62302#1513824>, @probinson wrote:

> Drive-by: For the "dead code" did you check whether gcc emits DW_AT_start_scope?  LLDB should support more than just what Clang emits.


I agree with that, but the fact that clang does not emit the attribute was not the reason why I called the code dead. It was the reason why I did not bother implementing it here. :)

The code is dead because it parses the ranges in the DW_AT_start_scope attribute, and then it just throws them away. What's missing is the part which converts this into the debug-format-agnostic form that the rest of lldb can use.


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

https://reviews.llvm.org/D62302