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

Ha!  It was apparently so easy to add the test that it wasn't noticeable (at 
least by me!).

This isn't directly relevant to your patch, since this wasn't behavior you 
changed, but it seems weird to me to suppress the commands but print their 
output?  Is that really the behavior we want for -Q?

Anyway, that's more fuel for thought than something you need to address in this 
patch, where you're just making existing behavior work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112988

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

Reply via email to