[PATCH] D50388: Respect PYTHONPATH
    Dan Liew via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Aug 22 01:27:25 PDT 2018
    
    
  
delcypher added inline comments.
================
Comment at: utils/lit/tests/lit.cfg:39
+pythonpath_list = [lit_path]
+if 'PYTHONPATH' in os.environ:
+    pythonpath_list.append(os.environ['PYTHONPATH'])
----------------
You should add a comment above the `if 'PYTHONPATH'` explaining why this is being here because it looks like the `Required because some tests import the lit module` comment is describing the entire block of code, whereas it actually only describes the very first statement (`pythonpath_list = [lit_path]`).
Repository:
  rL LLVM
https://reviews.llvm.org/D50388
    
    
More information about the llvm-commits
mailing list