bkramer added inline comments. ================ Comment at: include-fixer/IncludeFixer.cpp:104 @@ -103,3 @@ - // class Bar; - // Foo<Bar> foo; - // ---------------- Does this patch do the right thing for the test case in the comment? Otherwise we'll try add includes to system headers all the time, which both wrong and invisible to the user.
http://reviews.llvm.org/D20966 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits