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 committed May 14, 2019
1 parent 1584213 commit ce0da8b
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
Original file line number Diff line number Diff line change
@@ -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 ce0da8b

Please sign in to comment.