aaron.ballman added a comment. In http://reviews.llvm.org/D16786#341938, @Eugene.Zelenko wrote:
> I didn't notice test cases for included files for other checkers. So it's > hard to tell for should special test case introduced for this specific > problem or not. Generally speaking, all changes should have a test case accompanying them unless the change is NFC. It helps ensure we don't regress behavior the we want with the fix. Repository: rL LLVM http://reviews.llvm.org/D16786 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits