[lit] Fix a problem with spaces in the python path by adding quotes around it

[asmith]

Summary

This fixes two failing tests on Windows with an installed version of python that has spaces in the path.

• elf/lto/cache.ll
• mach-o/dependency_info.yaml

[zturner]

I wonder if it would be worth fixing this in lit? Seems like each substitution should be inherently quoted. Do you know where in lit this is actually failing? I only bring this up because this is the second such bug you've fixed of this nature, so if it's a recurring problem then we should perhaps fix it at the source. For example, what if %clang also has spaces in the path? Will that also fail?

[stella.stamenova]

I agree that making a change to lit to pass the quoted path is the "correct" change, but it's a more disruptive change (see below) and since only a handful of tests invoke %python, I think we should change the tests to pass quotes instead. That said, I'm happy to make the change either way to conform to standards.

Details:

I did some reading through the lit code and %clang will actually work correctly. It looks like python is the only one that is explicitly set rather than using one of the helper functions which set the path correctly. The explicit call for python does not:

self.config.substitutions.append(('%python', sys.executable))

(the code is here: llvm\utils\lit\lit\llvm\config.py)

Changing this to something like:

python_path = r"'{}'".format(sys.executable)
self.config.substitutions.append(('%python', python_path))

will fix the issues from the current change, however, there are tests that will start failing instead because they expect that the path does not have quotes around it. For example, there are two LLVM tests that start failing:

LLVM :: BugPoint/compile-custom.ll
LLVM :: BugPoint/unsymbolized.ll

I suspect there may be other tests in clang/LLD/LLDB. Note that one of these tests (unsymbolized.ll) has a trivial fix of removing the quotes around %python, but compile-custom.ll does not because the command it's using is more complicated:

; RUN: bugpoint -load %llvmshlibdir/BugpointPasses%shlibext --compile-custom --compile-command="%python %/s.py arg1 arg2" --opt-command opt --output-prefix %t %s | FileCheck %s

Since I've not run all of the possible lit tests yet, I am not sure whether any other failures will have non-trivial changes as well.

test/ELF/lto/cache.ll

 ; REQUIRES: x86
 
 ; RUN: opt -module-hash -module-summary %s -o %t.o
 ; RUN: opt -module-hash -module-summary %p/Inputs/cache.ll -o %t2.o
 
 ; RUN: rm -Rf %t.cache && mkdir %t.cache
 ; Create two files that would be removed by cache pruning due to age.
 ; We should only remove files matching the pattern "llvmcache-*".
 ; RUN: touch -t 197001011200 %t.cache/llvmcache-foo %t.cache/foo
 ; RUN: ld.lld --thinlto-cache-dir=%t.cache --thinlto-cache-policy prune_after=1h:prune_interval=0s -o %t3 %t2.o %t.o
 
 ; Two cached objects, plus a timestamp file and "foo", minus the file we removed.
 ; RUN: ls %t.cache | count 4
 
 ; Create a file of size 64KB.
-; RUN: %python -c "print(' ' * 65536)" > %t.cache/llvmcache-foo
+; RUN: "%python" -c "print(' ' * 65536)" > %t.cache/llvmcache-foo
 
 ; This should leave the file in place.
 ; RUN: ld.lld --thinlto-cache-dir=%t.cache --thinlto-cache-policy cache_size_bytes=128k:prune_interval=0s -o %t3 %t2.o %t.o
 ; RUN: ls %t.cache | count 5
 
 ; This should remove it.
 ; RUN: ld.lld --thinlto-cache-dir=%t.cache --thinlto-cache-policy cache_size_bytes=32k:prune_interval=0s -o %t3 %t2.o %t.o
 ; RUN: ls %t.cache | count 4

test/mach-o/dependency_info.yaml

 # Test -dependency_info option
 #
 # RUN: ld64.lld -arch x86_64 -test_file_usage  \
 # RUN:        -dependency_info %t.info \
 # RUN:        -path_exists /System/Library/Frameworks \
 # RUN:        -path_exists /System/Library/Frameworks/Foo.framework/Foo \
 # RUN:        -path_exists /Custom/Frameworks \
 # RUN:        -path_exists /Custom/Frameworks/Bar.framework/Bar \
 # RUN:        -F/Custom/Frameworks \
 # RUN:        -framework Bar \
 # RUN:        -framework Foo
-# RUN: %python %p/Inputs/DependencyDump.py %t.info | FileCheck %s
+# RUN: '%python' %p/Inputs/DependencyDump.py %t.info | FileCheck %s
 
 
 # CHECK: linker-vers: lld
 # CHECK: input-file:  /Custom/Frameworks{{[/\\]}}Bar.framework{{[/\\]}}Bar
 # CHECK: not-found:   /Custom/Frameworks{{[/\\]}}Foo.framework{{[/\\]}}Foo
 # CHECK: input-file:  /System/Library/Frameworks{{[/\\]}}Foo.framework{{[/\\]}}Foo
 # CHECK: output-file: a.out