[PATCH] Refactor Matcher<T> and DynTypedMatcher to reduce overhead of casts.

Samuel Benzaquen sbenza at google.com
Tue Sep 30 06:17:45 PDT 2014


Sorry about the breakage.
And thanks for the rollback!

On Mon, Sep 29, 2014 at 8:15 PM, NAKAMURA Takumi <geek4civic at gmail.com>
wrote:

> Excuse me, I reverted it in r218648, for now.
>
> - MSC17 doesn't compile it.
>
>   clang/include/clang/ASTMatchers/ASTMatchersInternal.h(387) : error
> C4519: default template arguments are only allowed on a class template
>
> - g++-4.7.2 reported so many warnings. See also
> http://bb.pgr.jp/builders/clang-3stage-x86_64-linux/builds/7720/steps/stage1_build
>
> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D5485
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140930/48f5cd67/attachment.html>


More information about the cfe-commits mailing list