bulbazord wrote:

> Currently this is a little hard to read given a bunch of shared setup 
> followed by two possible ways to read the TLS data. If those implementations 
> don't require too many inputs, it might be worth splitting this out into two 
> helper functions to make this easier to follow and extend in the future.

I can do that. Since this change is quite large as-is, perhaps I should break 
this up into smaller changes and submit them separately?

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

Reply via email to