[PATCH] D83432: [NFC][llvm-reduce] Don't `rm -rf` in tests, `rm -f` is enough
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 9 01:11:28 PDT 2020
lebedev.ri added a comment.
In D83432#2140532 <https://reviews.llvm.org/D83432#2140532>, @dblaikie wrote:
> If the test is writing the output file anyway - is the rm necessary? (lots of tests write to output files via "-o %t" from some tool or another and most don't delete %t before doing so)
They were added by you in rL372054 <https://reviews.llvm.org/rL372054>, so indeed the revert of that commit would be the best solution now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83432/new/
https://reviews.llvm.org/D83432
More information about the llvm-commits
mailing list