clayborg added a comment.

In D112279#3084758 <https://reviews.llvm.org/D112279#3084758>, @JDevlieghere 
wrote:

> Can we use the UUID as the module identifier (at least in the JSON)? Other 
> than that, this LGTM.

Some binaries don't have UUID values and I didn't want to use a UUID when it 
was there and something else when the UUID wasn't there. The main example is 
all .o files don't have UUID values and many linux binaries may or may not have 
them.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112279/new/

https://reviews.llvm.org/D112279

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to