[PATCH] D53729: [llvm-cov] Don't remap existing paths
David Greene via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 29 08:56:46 PDT 2018
greened added a comment.
> Maybe we could just add 'REQUIRES: build-dir-not-slash-tmp' or something to the affected lit tests.
It would be unfortunate to disable tests just because of this issue. I understand your point, though, and remapping always certainly has simpler semantics.
How difficult is it to add new REQUIRES clauses? I've never done it before.
Would it make sense to try to change the tests to not need remapping of /tmp? I don't know how I'd do that, so ideas are welcome.
Repository:
rL LLVM
https://reviews.llvm.org/D53729
More information about the llvm-commits
mailing list