Lgtm<br><div class="gmail_quote"><div dir="ltr">On Sat, Sep 30, 2017 at 2:17 PM Michał Górny 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">mgorny created this revision.<br>
Herald added a reviewer: modocache.<br>
<br>
Fix llvm_tools_dir attribute access not to fail when the variable is not<br>
present. This directory is not really necessary to run lit tests,<br>
and the code already accounts for it being None.<br>
<br>
The reference was added in <a href="https://reviews.llvm.org/rL313407" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL313407</a>, and it breaks the stand-alone lit<br>
package in Gentoo.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D38442" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38442</a><br>
<br>
Files:<br>
  utils/lit/tests/lit.cfg<br>
<br>
<br>
Index: utils/lit/tests/lit.cfg<br>
===================================================================<br>
--- utils/lit/tests/lit.cfg<br>
+++ utils/lit/tests/lit.cfg<br>
@@ -65,6 +65,6 @@<br>
     config.available_features.add('windows')<br>
<br>
 # Add llvm tools directory if this config is being loaded indirectly<br>
-if config.llvm_tools_dir is not None:<br>
+if getattr(config, 'llvm_tools_dir', None) is not None:<br>
     path = os.path.pathsep.join((config.llvm_tools_dir, config.environment['PATH']))<br>
     config.environment['PATH'] = path<br>
<br>
<br>
</blockquote></div>