[PATCH] D20597: Speed up check by using a recursive visitor.

Samuel Benzaquen via cfe-commits cfe-commits at lists.llvm.org
Tue May 24 13:51:25 PDT 2016


sbenza created this revision.
sbenza added a reviewer: alexfh.
sbenza added a subscriber: cfe-commits.

Use a recursive visitor instead of forEachDescendant() matcher.
The latter requires several layers of virtual function calls for each node and
it is more expensive than the visitor.
Benchmark results show improvement of ~6% walltime in clang-tidy.

http://reviews.llvm.org/D20597

Files:
  clang-tidy/readability/FunctionSizeCheck.cpp
  clang-tidy/readability/FunctionSizeCheck.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20597.58311.patch
Type: text/x-patch
Size: 6837 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160524/4d41fa0e/attachment-0001.bin>


More information about the cfe-commits mailing list