[Lldb-commits] [PATCH] D65611: [Driver] Expand the target in the driver.

Tue Aug 6 02:53:39 PDT 2019

labath added a comment.

In D65611#1615161 <https://reviews.llvm.org/D65611#1615161>, @JDevlieghere wrote:

> We definitely need the VFS change for the reproducers, no argument there. My earlier comment was more about the change in general, orthogonal to the reproducers.

Ah, I see. Thanks for explaining.

As for the expansion, I think a better way to handle that would be to change the output of the "target create" to show to resolved path, independent of the actual input:

  $ bin/lldb ls
  (lldb) target create "ls"
  Current executable set to 'ls' (x86_64).

We could change the last line to say "/bin/ls" without affecting the argv[0] of the target program. That way, you'd get the resolved path no matter how the user specifies the target to debug...

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

https://reviews.llvm.org/D65611