labath added a reviewer: jingham.
labath added a comment.

I don't have a hard objection to this, but if we're don't have a use for it, I 
don't see a need for adding it right now.
Particularly as it sounds weird to me that "user commands" are not a subset of 
"commands". I know that is how the underlying api works, but the underlying api 
looks weird to me too. However, that one (unlike the SB version) can be changed 
whenever we want.

As for the test, the same remarks I made on the other review apply here too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77582



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

Reply via email to