[PATCH] D23651: [clang-rename] improve performance for rename-all

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 18 02:49:42 PDT 2016


omtcyfz created this revision.
omtcyfz added reviewers: alexfh, vmiklos.
omtcyfz added a subscriber: cfe-commits.

As Miklos Vajna [[ http://lists.llvm.org/pipermail/cfe-dev/2016-August/050398.html | noticed ]] `clang-rename rename-all` has significant performance problems, which exposed the fact that clang-rename parses translation unit **N** times where **N** stands for the number of `{offset | old-name} -> new-name` pairs.

This patch prevents clang-rename from parsing translation unit multiple times.

https://reviews.llvm.org/D23651

Files:
  clang-rename/USRFindingAction.cpp
  clang-rename/USRFindingAction.h
  clang-rename/tool/ClangRename.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23651.68502.patch
Type: text/x-patch
Size: 9955 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160818/a615fd72/attachment-0001.bin>


More information about the cfe-commits mailing list