================
@@ -564,9 +557,8 @@ class UnwindPlan {
   }
 
 private:
-  typedef std::vector<RowSP> collection;
-  collection m_row_list;
-  AddressRange m_plan_valid_address_range;
+  std::map<lldb::addr_t, RowSP> m_rows;
----------------
labath wrote:

It won't work, since the order of entries is important. What I can imagine is 
constructing an unwind plan (in lower case, i.e. not  `UnwindPlan`) in a map 
object, and then passing that to an UnwindPlan constructor which will flatten 
it in. It would be a slightly larger change, but I think the result could be 
nice as it would essentially make an UnwindPlan immutable after construction. 
(I like immutable data structures, and unwind plans really must be immutable as 
they are cached)

https://github.com/llvm/llvm-project/pull/127661
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to