[Lldb-commits] [PATCH] D91155: [lldb] Fix TestErrorMessages test on standalone builds by adding lldb-server substitution

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 10 06:53:37 PST 2020


teemperor added inline comments.
Herald added a subscriber: JDevlieghere.


================
Comment at: lldb/test/Shell/helper/toolchain.py:65-68
         ToolSubst('%platformserver',
                   command=FindTool('lldb-server'),
                   extra_args=['platform'],
                   unresolved='ignore'),
----------------
labath wrote:
> I guess we don't need this then, as one can just write `%lldb-server platform`. It also seems to be unused.
It seems the test that was using this substitution was accidentally deleted in the lldb-mi removal, so I reinstated that test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91155



More information about the lldb-commits mailing list