[PATCH] D26734: [ELF] Don't replace path separators on *NIX

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 11:15:17 PST 2016


ruiu added inline comments.


================
Comment at: test/ELF/reproduce-backslash.s:5
+# legal for a filename to contain backslashes.
+# RUN: rm -rf %t.dir
+# RUN: mkdir -p %t.dir/build
----------------
ruiu wrote:
> I don't think you need to to create a directory.
And I want to avoid "rm -rf" in general because one mistake can really be disastrous (not only one person but everybody who checks out a buggy test file and run it).


https://reviews.llvm.org/D26734





More information about the llvm-commits mailing list