JDevlieghere wrote:

> I pushed a new commit to simplify the comments.

Thanks!

> Additionally, does it make sense to add a file-local/private template helper 
> `GetUInt<T>`, and make the existing public `GetU16/32/64` call it?

I think so, llvm generally favors small static helpers.


https://github.com/llvm/llvm-project/pull/213038
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to