[PATCH] Header dependencies support for modularize
klimek at google.com
Thu Aug 22 01:21:28 PDT 2013
On Wed, Aug 21, 2013 at 10:20 PM, Kim Gräsman <kim.grasman at gmail.com> wrote:
> Hi Manuel,
> On Wed, Aug 21, 2013 at 3:15 PM, Manuel Klimek <klimek at google.com> wrote:
> > On Wed, Aug 21, 2013 at 3:06 PM, Kim Gräsman <kim.grasman at gmail.com>
> >> I've wanted to add compilation database support, but the way tooling
> >> is wired it seems I need to extract compilation db and source paths
> >> from the command-line and pass them to ClangTool::ClangTool. The
> >> compilation db is (sort of) handled by FixedCompilationDatabase, but
> >> it seems the source paths are assumed to be tool arguments, not Clang
> >> arguments.
> > Use a clang plugin if you want to run as part of a build. The code that
> > your main logic is basically the same, and you can use ASTMatchers and
> > stuff from tooling in your clang plugin just fine.
> Thanks, but this tool already exists:
> and we'd like to have it act either as part of a build or stand-alone
> with a compilation database. It seems hard to combine the two,
it seems like it should be fairly straight-forward to pull out the main
method into its own .cc file, and have 2 .cc files:
iwyu_main.cc (the current main code)
iwyu_plugin.cc (a clang plugin that hands a new IwyuAction to clang to run)
Please let me know if I'm missing something...
> I'm interested in the Tooling approach because it would cut lots of
> boilerplate from our setup and make us less exposed to driver changes.
> - Kim
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits