[PATCH] D52273: [clangd] Initial implementation of expected types
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 9 08:14:16 PST 2018
ilya-biryukov added a comment.
I've run the measurements on a highly simplified vs the original complicated model and got roughly the same results wrt to ranking improvements, so sending a new version of the patch with highly simplified mode for the type representation.
I believe there are still gains to be had from a more thorough treatment of C++ conversions, but there is definitely much to do in other areas that should provide better ground for seeing the actual improvements with the more complicated model.
In any case, starting with something simple is definitely a better ground. Thanks for initial review and suggestions!
And please take a look at the new version, it is significantly simpler and should be pretty easy to review :-)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52273
More information about the cfe-commits
mailing list