mstorsjo added a comment.

In https://reviews.llvm.org/D45505#1074744, @chandlerc wrote:

> Definitely need tests here.


Do you have any suggestion on where to create them and what kind? Just a normal 
test with a fake directory tree, or some sort of unit test for the gcc version 
number matching logic? (And if creating a unit test, how does that work since 
this class isn't in the public include directory?)


Repository:
  rC Clang

https://reviews.llvm.org/D45505



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to