maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Orc][examples] Add lit ToolSubst for LLJITWithRemoteDebugging example 

The test case added in 258f055ed9 was lacking two important details for the test infrastructure. ae217bf1f3 added the executable to LLVM_TEST_DEPENDS in CMake to make sure the exectubale gets built before we run the test suite. This patch adds a ToolSubst for the executable in LIT, which replaces the tool invokation in the RUN line with an absolute path. It makes sure we don't run accidentally run some other tool from the user's PATH. The test works without it in case LLVM's main binary directory happens to be the working directory (which is default apparently). Configurations that don't build the examples ignore failures for this ToolSubst (and won't run the test).

Reviewed By: echristo

Differential Revision: https://reviews.llvm.org/D99931
This commit is contained in:
Stefan Gränitz 2021-04-07 09:44:18 +02:00
parent 03f7b13d44
commit 030ac786d4
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
llvm/test/lit.cfg.py
View File

@ -177,7 +177,8 @@ tools.extend([
ToolSubst('Kaleidoscope-Ch6', unresolved='ignore'), ToolSubst('Kaleidoscope-Ch6', unresolved='ignore'),
ToolSubst('Kaleidoscope-Ch7', unresolved='ignore'), ToolSubst('Kaleidoscope-Ch7', unresolved='ignore'),
ToolSubst('Kaleidoscope-Ch8', unresolved='ignore'), ToolSubst('Kaleidoscope-Ch8', unresolved='ignore'),
ToolSubst('LLJITWithThinLTOSummaries', unresolved='ignore')]) ToolSubst('LLJITWithThinLTOSummaries', unresolved='ignore'),
ToolSubst('LLJITWithRemoteDebugging', unresolved='ignore')])
llvm_config.add_tool_substitutions(tools, config.llvm_tools_dir) llvm_config.add_tool_substitutions(tools, config.llvm_tools_dir)