jasonmolenda accepted this revision.
jasonmolenda added a comment.
This revision is now accepted and ready to land.

LGTM.  Putting this in `register info` is good, it's not too easily 
discoverable (to be honest I didn't know this command existed today), but as 
you say, the number of people who want to see this are relatively small so 
putting it in a more mainstream command like register read may not be the best 
choice.  We can start with only in `register info`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152918

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

Reply via email to