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

It is a bit odd to follow the what the help summary says is a valid option set 
and then get an error.   But this is a corner case, and since you say 
explicitly that you can't provide both that's probably good enough.  Would you 
mind changing "Incompatible with" to: "Cannot be specified at the same time 
as".  Incompatible isn't precise - does it mean you can't use them both, or 
that if you do something weird will happen?

Other than that, this is fine.  Thanks for working on this.


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

https://reviews.llvm.org/D137000

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

Reply via email to