[Lldb-commits] [lldb] [lldb/Interpreter] Fix ambiguous partial command resolution (PR #101934)

Med Ismail Bennani via lldb-commits lldb-commits at lists.llvm.org
Mon Aug 5 10:49:27 PDT 2024


medismailben wrote:

> This is for this sort of lookup:
> 
> ```
> (lldb) reg read pc
> ```
> 
> Right? As opposed to tab completion.

Right, tab completion already can suggest multiple candidates, so that worked already.

> This needs a test case or at least a clear example as a comment in the code if a test is not possible. It seems fine from how you've described it but these things come across better with a clear example.

Sorry if this wasn't clear, but I didn't write a test case because there was already a test case in https://github.com/llvm/llvm-project/pull/97263, but I guess I could write a test to test the ordering precisely (custom command, then alias, then native command)

https://github.com/llvm/llvm-project/pull/101934


More information about the lldb-commits mailing list