[PATCH] [clang-tidy] Move -extra-arg handling to CommonOptionsProvider

Alexander Kornienko alexfh at google.com
Mon Nov 3 11:58:26 PST 2014


The previous version looks marginally better to me due to slightly less
code, but this one turned out to be better than I expected. Which one do
you prefer?
On 3 Nov 2014 11:35, "Alexander Kornienko" <alexfh at google.com> wrote:

> Make ArgumentsAdjustingCompilations more generic by using
> ArgumentsAdjusters.
>
> http://reviews.llvm.org/D6073
>
> Files:
>   include/clang/Tooling/ArgumentsAdjusters.h
>   include/clang/Tooling/CommonOptionsParser.h
>   lib/Tooling/ArgumentsAdjusters.cpp
>   lib/Tooling/CommonOptionsParser.cpp
>   tools/clang-check/ClangCheck.cpp
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141103/efcdca18/attachment.html>


More information about the cfe-commits mailing list