Fine by me, can you also update the 'lit' docs in CommandGuide to note this.<span></span><br><br>On Monday, May 4, 2015, Matthias Braun <<a href="mailto:matze@braunis.de">matze@braunis.de</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi ddunbar,<br>
<br>
These changes allow usages where you want to pass an additional<br>
commandline option to all invocations of a specific llvm tool. Example:<br>
<br>
> llvm-lit -Dllc=llc -enable-misched -verify-machineinstrs<br>
<br>
REPOSITORY<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D9487" target="_blank">http://reviews.llvm.org/D9487</a><br>
<br>
Files:<br>
  test/lit.cfg<br>
  utils/lit/lit/main.py<br>
<br>
Index: test/lit.cfg<br>
===================================================================<br>
--- test/lit.cfg<br>
+++ test/lit.cfg<br>
@@ -267,8 +267,12 @@<br>
                           pattern)<br>
     tool_pipe = tool_match.group(2)<br>
     tool_name = tool_match.group(4)<br>
-    tool_path = lit.util.which(tool_name, llvm_tools_dir)<br>
-    if not tool_path:<br>
+    # Did the user specify the tool path + arguments? This allows things like<br>
+    # llvm-lit "-Dllc=llc -enable-misched -verify-machineinstrs"<br>
+    tool_path = lit_config.params.get(tool_name)<br>
+    if tool_path is None:<br>
+        tool_path = lit.util.which(tool_name, llvm_tools_dir)<br>
+    if tool_path is None:<br>
         # Warn, but still provide a substitution.<br>
         lit_config.note('Did not find ' + tool_name + ' in ' + llvm_tools_dir)<br>
         tool_path = llvm_tools_dir + '/' + tool_name<br>
Index: utils/lit/lit/main.py<br>
===================================================================<br>
--- utils/lit/lit/main.py<br>
+++ utils/lit/lit/main.py<br>
@@ -146,7 +146,7 @@<br>
     parser.add_option("", "--config-prefix", dest="configPrefix",<br>
                       metavar="NAME", help="Prefix for 'lit' config files",<br>
                       action="store", default=None)<br>
-    parser.add_option("", "--param", dest="userParameters",<br>
+    parser.add_option("-D", "--param", dest="userParameters",<br>
                       metavar="NAME=VAL",<br>
                       help="Add 'NAME' = 'VAL' to the user defined parameters",<br>
                       type=str, action="append", default=[])<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote>