Skip to content

Commit

Permalink
[test]Make test work on Windows
Browse files Browse the repository at this point in the history
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.

Reviewed by: kzhuravl, grimar

Differential Revision: https://reviews.llvm.org/D61856

llvm-svn: 360660
  • Loading branch information
jh7370 authored and MrSidims committed May 17, 2019
1 parent c226c9b commit 1c323fe
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions llvm/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
Expand Down

0 comments on commit 1c323fe

Please sign in to comment.