diff options
author | Sam McCall <sam.mccall@gmail.com> | 2019-11-13 20:16:23 +0100 |
---|---|---|
committer | Sam McCall <sam.mccall@gmail.com> | 2019-11-14 17:35:00 +0100 |
commit | bbcbb10e2d0848b611c51b1ff56c758645c55f3b (patch) | |
tree | b656a3493180d6a2fc342080676538c9ca2c9bf4 /clang/test/Modules | |
parent | 37abeed8145b5faf02ca9ae3ffedf46b52947f41 (diff) | |
download | bcm5719-llvm-bbcbb10e2d0848b611c51b1ff56c758645c55f3b.tar.gz bcm5719-llvm-bbcbb10e2d0848b611c51b1ff56c758645c55f3b.zip |
[clangd] Ignore more implicit nodes in computing selection.
Summary:
The DeclRefExpr for the callee of overloaded `operator()` and `operator[]` are
assigned the range of the paren/bracket lists in the AST.
These are better thought of as implicit (at least `()` - `[] is murkier).
But there's no bit on Expr for implicit, so just ignore them on our side.
While here, deal with the case where an implicit stmt (e.g. implicit-this)
is wrapped in an implicit cast. Previously we ignored the statement but not
the cast, and so the cast ended up being selected.
Fixes https://github.com/clangd/clangd/issues/195
Reviewers: kadircet, lh123
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70194
Diffstat (limited to 'clang/test/Modules')
0 files changed, 0 insertions, 0 deletions