labath wrote:

The implementation of Variable::IsThreadLocal is most likely a non-starter, as 
it introduces a relatively expensive operation on the common variable update 
path for all variables. Doing this (once) at variable creation would be better, 
but I still have very big reservations about this approach. AFAICT, there's no 
way to guarantee that the variable you find this way will be the one that is 
actually described by the DWARF. You could even have more than one variable 
with the same name if they're `static thread_local`.

This really sounds like something where the compiler should give us more to go 
on -- instead of us trying to divine this information out of thin air.

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

Reply via email to