[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