malaperle added a comment. In https://reviews.llvm.org/D52089#1235851, @ioeric wrote:
> In https://reviews.llvm.org/D52089#1235777, @malaperle wrote: > > > why? > > > I wanted to get some numbers and update the patch summary, but somehow > forgot. Sorry about that and thanks for asking! > > The AST matcher pops up in performance profile. Although not the most > expensive thing we could optimize, this still brings `indexAST` latency from > ~5.3s to ~4.4s for a large TU with big preamble. Thanks! That's what I was looking for :) Sounds good to me. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52089 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
