[PATCH] Remove VariadicOperatorMatcherInterface as it is redundant with logic from DynTypedMatcher.

Samuel Benzaquen sbenza at google.com
Fri Nov 14 13:11:26 PST 2014


Hi klimek,

The generic variadic matcher is faster (one less virtual function call
per match) and doesn't require template instantiations which reduces
compile time and binary size.
Registry.cpp.o generates ~14% less symbols and compiles ~7.5% faster.
The change also speeds up our clang-tidy benchmark by ~2%.

http://reviews.llvm.org/D6278

Files:
  include/clang/ASTMatchers/ASTMatchersInternal.h
  include/clang/ASTMatchers/Dynamic/VariantValue.h
  lib/ASTMatchers/ASTMatchersInternal.cpp
  lib/ASTMatchers/Dynamic/Marshallers.h
  lib/ASTMatchers/Dynamic/VariantValue.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6278.16247.patch
Type: text/x-patch
Size: 8763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141114/7ffda022/attachment.bin>


More information about the cfe-commits mailing list