================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.cpp:23
@@ +22,3 @@
+
+static cl::opt<std::string> CLIncludePaths(
+ "include", cl::desc("Comma seperated list of filepaths to consider to be "
----------------
Manuel Klimek wrote:
> Just FYI: I personally have a strong dislike of putting command line
> arguments into library files - it means that every binary that happens to
> link in this file transitively will see this argument, which might not be a
> problem now, but in my experience it always leads to headaches down the road.
> What I'd do instead is: put the command line args into the main file, and
> have a constructor for IncludeExcludeInfo that takes the lists as arguments.
Maybe mark the command-line args hidden too for now since the information they
provide is not actually used for anything yet in cpp11-migrate.
http://llvm-reviews.chandlerc.com/D648
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
