<div><div dir="auto">+mgorny</div><br><div class="gmail_quote"><div>On Sat, Nov 18, 2017 at 2:24 PM Eric Fiselier via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">EricWF created this revision.<br>
Herald added a reviewer: modocache.<br>
<br>
Currently, LIT configures the LLVM binary path before the Clang binary path. However this breaks testing out-of-tree Clang builds (where the LLVM binary path includes a copy of Clang).<br>
<br>
This patch reverses the order of the paths when looking for Clang, putting the Clang binary directory first.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D40217" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40217</a><br>
<br>
Files:<br>
  utils/lit/lit/llvm/config.py<br>
<br>
<br>
Index: utils/lit/lit/llvm/config.py<br>
===================================================================<br>
--- utils/lit/lit/llvm/config.py<br>
+++ utils/lit/lit/llvm/config.py<br>
@@ -369,8 +369,9 @@<br>
         # Tweak the PATH to include the tools dir and the scripts dir.<br>
         paths = [self.config.llvm_tools_dir]<br>
         tools = getattr(self.config, 'clang_tools_dir', None)<br>
+        # Put Clang first to avoid LLVM from overriding out-of-tree clang builds.<br>
         if tools:<br>
-            paths = paths + [tools]<br>
+            paths = [tools] + paths<br>
         self.with_environment('PATH', paths, append_path=True)<br>
<br>
         paths = [self.config.llvm_shlib_dir, self.config.llvm_libs_dir]<br>
<br>
<br>
</blockquote></div></div>