[PATCH] D61856: [test]Make test work on Windows

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 06:08:36 PDT 2019


jhenderson created this revision.
jhenderson added reviewers: khemant, aprantl, echristo, grimar, kzhuravl.
Herald added a subscriber: rupprecht.
Herald added a project: LLVM.

Previously, the test didn't work because '\' characters appeared in the sed string, causing bogus escape characters to form in the substituted string literal. Switching to using '%/p' causes the path to be emitted with '/' characters instead, so that there are are no escaping issues.


Repository:
  rL LLVM

https://reviews.llvm.org/D61856

Files:
  test/tools/llvm-objdump/X86/source-interleave-x86_64.ll


Index: test/tools/llvm-objdump/X86/source-interleave-x86_64.ll
===================================================================
--- test/tools/llvm-objdump/X86/source-interleave-x86_64.ll
+++ test/tools/llvm-objdump/X86/source-interleave-x86_64.ll
@@ -1,5 +1,4 @@
-;  REQUIRES: shell
-;  RUN: sed -e "s,SRC_COMPDIR,%p/Inputs,g" %s > %t.ll
+;  RUN: sed -e "s,SRC_COMPDIR,%/p/Inputs,g" %s > %t.ll
 ;  RUN: llc  -o %t.o -filetype=obj -mtriple=x86_64-pc-linux  %t.ll
 ;  RUN: llvm-objdump -d -l %t.o >%t0
 ;  RUN: llvm-objdump -dl %t.o >%t1


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61856.199260.patch
Type: text/x-patch
Size: 540 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190513/5703ba7d/attachment.bin>


More information about the llvm-commits mailing list