[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