[PATCH] D61103: [clang] Add tryToAttachCommentsToDecls method to ASTContext
Jan Korous via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 24 16:33:41 PDT 2019
jkorous created this revision.
jkorous added reviewers: gribozavr, arphaman.
Herald added subscribers: cfe-commits, dexonsmith.
Herald added a project: clang.
Loading external comments and sorting them is expensive - mostly due to getDecomposedLoc() begin expensive. For modules with very large number of comments (~100k) this is prohibitively expensive.
In this particular case we are actually not at all interested in getting comments for declarations - just using a side-effect of the implementation which causes documentation comments to be parsed (doxygen) and attached to relevant declarations.
The FIXME in tests is fixed now.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 7892 bytes
Desc: not available
More information about the cfe-commits