[PATCH] D29185: [lit] Allow llvm's build and test systems to support paths with spaces

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 15 05:04:12 PST 2017


mgorny added a comment.

I would like to point out that if you proactively reject all paths with spaces, you wouldn't be able to add additional tests that would test support for paths with spaces ;-). While I agree that we should avoid spaces whenever possible, I think it would be reasonable to add tests for paths with spaces wherever we expect them to be well-supported and want to avoid regressions.


https://reviews.llvm.org/D29185





More information about the llvm-commits mailing list