sammccall added a comment.

In D61537#1491649 <https://reviews.llvm.org/D61537#1491649>, @ilya-biryukov 
wrote:

> Nice! Looking at the numbers, the improvements look worthwhile.
>
> The stopwords don't seem to be in the patch anymore, but they do seem like a 
> good idea. Specifically, we might want to remove all keywords - the intuition 
> is that they don't provide much value, because users **have** to put them.


The simplest versions of this didn't show any actual numerical benefit. There 
may be gains to be had by running the real lexer, will leave that for future 
work.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D61537



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to