[PATCH] Transform all files in a compilation database if no source files are provided.
Ariel Bernal
ariel.j.bernal at intel.com
Tue Aug 27 07:54:23 PDT 2013
At first I added an extra pair of -include/exclude and -include-from/exclude-from options but then I realized that for most of the cases this should be ok and easier to use.
I see your point on priming IncludeExcludeInfo if no -include/exclude was specified. Should we then enable -headers by default?
================
Comment at: cpp11-migrate/tool/Cpp11Migrate.cpp:257
@@ +256,3 @@
+ // if the file is included when include/exclude paths are specified.
+ if (IncludePaths.empty() ||
+ GlobalOptions.ModifiableFiles.isFileIncluded(*I))
----------------
Isn't this enough to test if include paths is empty? (Although it should be moved out the loop).
http://llvm-reviews.chandlerc.com/D1517
More information about the cfe-commits
mailing list