[cfe-commits] [PATCH] Implements support to run standalone tools
hfinkel at anl.gov
Tue Jan 24 07:43:04 PST 2012
On Tue, 2012-01-24 at 11:03 +0100, Manuel Klimek wrote:
> The attached patch adds support to run clang tools (FrontendActions)
> as standalone tools, or repeatedly in-memory in a process.
> This is useful for unit testing, map-reduce-style applications, source
> transformation daemons, and command line tools.
I am also interested in having this kind of functionality. A few quick
1. The coding standards say that function names should begin with a
2. The comments contain several references to CMake; what, if anything,
in this patch is tied to CMake, or designed to be compatible with CMake?
> +/// \param JsonDatabase A JSON formatted list of compile commands.
> This lookup
> +/// command supports only a subset of the JSON standard as written by
Please be more verbose here. What is not supported?
Generally, I think that it would be helpful for you to provide a
paragraph or two explaining how this extension is to be used, what kind
of things can be specified in JSON inputs, how this ties into CMake (or
not), etc. with a few small examples. Some of this can be gleaned from
the test case, but some nicely-formatted text (without all of the
escaping) would, IMHO, be earlier to read.
> Rietveld link:
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
Leadership Computing Facility
Argonne National Laboratory
More information about the cfe-commits