[PATCH] Add a clang-tidy flag to support temporary destructor-aware analysis (workaround for bug 15599).

Alexander Kornienko alexfh at google.com
Wed Apr 30 07:58:44 PDT 2014


On Wed, Apr 30, 2014 at 4:26 PM, Alex McCarthy <alexmc at google.com> wrote:

> ================
> Comment at: test/clang-tidy/temporaries.cpp:3
> @@ +2,3 @@
> +// FileCheck complains if the input file is empty, so add a dummy line.
> +// RUN: echo foo >> %t.log
> +// RUN: FileCheck %s < %t.log
> ----------------
> Alexander Kornienko wrote:
> > You don't need this now as the correct output is guaranteed to be
> non-empty. And you can now just pipe clang-tidy output to FileCheck
> Good catch: fixed in llvm-reviews.chandlerc.com/rL207653
>
> http://reviews.llvm.org/D3556
>
>
BTW, if you use Phabricator's arc tool, you can also use "arc commit",
which automatically closes the Differential revision in Phab.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140430/b8057b92/attachment.html>


More information about the cfe-commits mailing list