[cfe-commits] [PATCH] Implements support to run standalone tools
klimek at google.com
Wed Jan 25 06:59:13 PST 2012
On Tue, Jan 24, 2012 at 4:43 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> 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
> lower-case letter.
Done. I jumped on the opportunity to dogfood refactoring support in
our current tooling branch and wrote a script that changed all
incorrectly named functions automatically (and created a sed-script to
post-produce comment changes, which made me notice a bug in 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.
Hopefully better expressed now. Please let me know if you want more /
Thanks a lot for the review!
>> Rietveld link:
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
> Hal Finkel
> Postdoctoral Appointee
> Leadership Computing Facility
> Argonne National Laboratory
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 42069 bytes
Desc: not available
More information about the cfe-commits