Lgtm<br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 24, 2018 at 5:11 PM Stella Stamenova 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">stella.stamenova created this revision.<br>
stella.stamenova added reviewers: asmith, inglorion.<br>
Herald added subscribers: llvm-commits, dexonsmith, steven_wu, eraman, mehdi_amini.<br>
<br>
We needed quotes around %python before to make python work correctly (on Windows) if the path contains spaces. I recently made a change so that %python now inherently has quotes, so now adding quotes around %python makes the test fail because the quotes cancel each other.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D51244" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51244</a><br>
<br>
Files:<br>
  test/ThinLTO/X86/cache.ll<br>
<br>
<br>
Index: test/ThinLTO/X86/cache.ll<br>
===================================================================<br>
--- test/ThinLTO/X86/cache.ll<br>
+++ test/ThinLTO/X86/cache.ll<br>
@@ -106,14 +106,14 @@<br>
 ; RUN: rm -Rf %t.cache && mkdir %t.cache<br>
 ; Create cache files with different sizes.<br>
 ; Only 8B and 76B files should stay after pruning.<br>
-; RUN: "%python" -c "with open(r'%t.cache/llvmcache-foo-100k', 'w') as file: file.truncate(102400)"<br>
+; RUN: %python -c "with open(r'%t.cache/llvmcache-foo-100k', 'w') as file: file.truncate(102400)"<br>
 ; RUN: touch -t 198002011200 %t.cache/llvmcache-foo-100k<br>
-; RUN: "%python" -c "with open(r'%t.cache/llvmcache-foo-16', 'w') as file: file.truncate(16)"<br>
+; RUN: %python -c "with open(r'%t.cache/llvmcache-foo-16', 'w') as file: file.truncate(16)"<br>
 ; RUN: touch -t 198002021200 %t.cache/llvmcache-foo-16<br>
-; RUN: "%python" -c "with open(r'%t.cache/llvmcache-foo-77k', 'w') as file: file.truncate(78848)"<br>
+; RUN: %python -c "with open(r'%t.cache/llvmcache-foo-77k', 'w') as file: file.truncate(78848)"<br>
 ; RUN: touch -t 198002031200 %t.cache/llvmcache-foo-77k<br>
-; RUN: "%python" -c "with open(r'%t.cache/llvmcache-foo-8', 'w') as file: file.truncate(8)"<br>
-; RUN: "%python" -c "with open(r'%t.cache/llvmcache-foo-76', 'w') as file: file.truncate(76)"<br>
+; RUN: %python -c "with open(r'%t.cache/llvmcache-foo-8', 'w') as file: file.truncate(8)"<br>
+; RUN: %python -c "with open(r'%t.cache/llvmcache-foo-76', 'w') as file: file.truncate(76)"<br>
 ; RUN: llvm-lto -thinlto-action=run -exported-symbol=globalfunc %t2.bc %t.bc -thinlto-cache-dir %t.cache --thinlto-cache-max-size-bytes 78847<br>
 ; RUN: ls %t.cache/llvmcache-foo-8<br>
 ; RUN: ls %t.cache/llvmcache-foo-76<br>
@@ -126,14 +126,14 @@<br>
 ; RUN: rm -Rf %t.cache && mkdir %t.cache<br>
 ; Create cache files with different sizes.<br>
 ; Only 75B and 76B files should stay after pruning.<br>
-; RUN: "%python" -c "print(' ' * 1023)" > %t.cache/llvmcache-foo-1023<br>
+; RUN: %python -c "print(' ' * 1023)" > %t.cache/llvmcache-foo-1023<br>
 ; RUN: touch -t 198002011200 %t.cache/llvmcache-foo-1023<br>
-; RUN: "%python" -c "print(' ' * 15)" > %t.cache/llvmcache-foo-15<br>
+; RUN: %python -c "print(' ' * 15)" > %t.cache/llvmcache-foo-15<br>
 ; RUN: touch -t 198002021200 %t.cache/llvmcache-foo-15<br>
-; RUN: "%python" -c "print(' ' * 7)" > %t.cache/llvmcache-foo-7<br>
+; RUN: %python -c "print(' ' * 7)" > %t.cache/llvmcache-foo-7<br>
 ; RUN: touch -t 198002031200 %t.cache/llvmcache-foo-7<br>
-; RUN: "%python" -c "print(' ' * 75)" > %t.cache/llvmcache-foo-75<br>
-; RUN: "%python" -c "print(' ' * 76)" > %t.cache/llvmcache-foo-76<br>
+; RUN: %python -c "print(' ' * 75)" > %t.cache/llvmcache-foo-75<br>
+; RUN: %python -c "print(' ' * 76)" > %t.cache/llvmcache-foo-76<br>
 ; RUN: llvm-lto -thinlto-action=run -exported-symbol=globalfunc %t2.bc %t.bc -thinlto-cache-dir %t.cache --thinlto-cache-max-size-files 4<br>
 ; RUN: ls %t.cache/llvmcache-foo-75<br>
 ; RUN: ls %t.cache/llvmcache-foo-76<br>
<br>
<br>
</blockquote></div>