[PATCH] D51648: [UBSan] Partially fix `test/ubsan/TestCases/Misc/log-path_test.cc` so that it can run on devices.

George Karpenkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 4 11:39:02 PDT 2018


george.karpenkov added inline comments.


================
Comment at: test/sanitizer_common/ios_commands/iossim_run.py:33
+    else:
+      rm_args_expanded.append(arg)
+  rm_cmd_line = ["/bin/rm"] + rm_args_expanded
----------------
Why not just use `subprocess.call(.., shell=True)` and let shell do the expansion?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D51648





More information about the llvm-commits mailing list