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

Reply via email to