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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to