zturner added a comment. In D70553#1757862 <https://reviews.llvm.org/D70553#1757862>, @aaron.ballman wrote:
> Can you add a test case for this functionality? The reason there's no test case is because it seems like that wouldn't really be any different than testing the functionality of `llvm::sys::fs`, which should already be handled at the llvm/Support layer. There's not really any interesting logic here. I can still do one if you think it's important but that was my reasoning anyway. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70553/new/ https://reviews.llvm.org/D70553 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits