[PATCH] D20966: [include-fixer] Add the missing header to the file where the unidentified symbol comes from.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 6 02:35:49 PDT 2016


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





More information about the cfe-commits mailing list