[PATCH] D55658: Avoid Decl::getASTContext in hot paths where possible, Part 1

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 15 10:32:16 PST 2018


vsk added a comment.

Thanks for working on this :). It’d be interesting to see some pre/post patch compile time numbers on CTMark.

Naive/strawman alternative here, but: what’s the impact on peak RSS and compile time of just storing an ASTContext pointer in Decl/DeclContext? If it’s not out of the question to grow Decl etc. and it gives a nice speed up, that might be a simpler approach.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55658/new/

https://reviews.llvm.org/D55658





More information about the cfe-commits mailing list