[Lldb-commits] [PATCH] D17449: Handle the case when a variable is only valid in part of the enclosing scope

Tamas Berghammer via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 23 10:15:25 PST 2016


tberghammer added a comment.

I can live with that but I still think the scope information belongs to the variable and not to the DWARF expression representing it's location. As far as I see keeping the information inside the variable have lower memory impact (assuming an empty list means full scope) because we have less Variable object then DWARFExpression object and it isn't complicate other use cases of DWARFExpression where scope isn't used at all.

So all in all I can move the information to the DWARFExpression but I think keeping them in the Variable class is better. Let me know what do you think.


http://reviews.llvm.org/D17449





More information about the lldb-commits mailing list