[Lldb-commits] [lldb] [lldb] [disassembler] chore: enhance VariableAnnotator to return structured data (PR #165163)

[via lldb-commits]

================
@@ -574,16 +589,39 @@ class VariableAnnotator {
     std::string name;
     /// Last printed location (empty means <undef>).
     std::string last_loc;
+    /// Address range where this variable state is valid.
+    lldb::addr_t start_address;
+    lldb::addr_t end_address;
+    /// Register numbering scheme for location interpretation.
+    lldb::RegisterKind register_kind;
+
+    std::optional<std::string> decl_file;
+    std::optional<uint32_t> decl_line;
+    std::optional<std::string> type_name;
----------------
n2h9 wrote:

Yes,  I agree. The idea was to have different structs for internal and external usage to have flexibility to have different structure of data. But probably this not necessary at all. 

Updated ✅ , thank you 🙇‍♀️.

https://github.com/llvm/llvm-project/pull/165163