[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