[Lldb-commits] [PATCH] D17724: Replace getopt with llvm::cl in lldb driver

[Greg Clayton via lldb-commits]

clayborg added a comment.

In http://reviews.llvm.org/D17724#364454, @zturner wrote:

> long and short options are supported, but the one I'm not sure about is the
>  case where you use a short option with no space.  Your example "-ax86_64"
>  might not work.  It might, just that it should be tested.  I'm 99%
>  confident the rest of them all work.
>
> Also not sure about this example: "% lldb /bin/ls --arch=x86_64 -- -lAF"
>
> If I understand correctly, this runs LLDB with the --arch=x86_64 argument,
>  and specifies the program to debug as "/bin/ls -IAF"?


Yes.

> Seems like a confusing syntax to me, is there any other way to specify this

>  currently?  like:

> 

> % lldb "/bin/ls -IAF" --arch=x86_64

>  % lldb /bin/ls --arch-x86_64 --args=-IAF

> 

> Or something along those lines?


Not sure, I am just pointing out what is going to change for people by making this switch due to the way getopt_long works.

For LLVM you might need to do this:

% lldb --arch=x86_64 -- /bin/ls -lAF

All args might need to be past all of the options and after the "--". I just don't know and was pointing out the differences people are likely to be confused by.


http://reviews.llvm.org/D17724