[PATCH] D42895: [libclang] Add `CXSymbolRole role` to CXIdxEntityRefInfo

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 6 10:24:48 PST 2018


MaskRay added a comment.

In https://reviews.llvm.org/D42895#998617, @yvvan wrote:

> I feel quite ok about this patch.
>
> Can you please add unit-tests?


Added

  // CHECK:      [indexEntityReference]: kind: field | name: y | {{.*}} | loc: 70:5 | {{.*}} | role: ref write
  // CHECK:      [indexEntityReference]: kind: field | name: x | {{.*}} | loc: 70:11 | {{.*}} | role: ref read
  // CHECK:      [indexEntityReference]: kind: function | name: foo3 | {{.*}} | loc: 71:10 | {{.*}} | role: ref addressof
  // CHECK:      [indexEntityReference]: kind: function | name: foo4 | {{.*}} | loc: 72:3 | {{.*}} | role: ref call

to `clang/test/Index/index-refs.cpp`. I will implement read/write after this revision is landed.. https://github.com/cquery-project/cquery/issues/262


Repository:
  rC Clang

https://reviews.llvm.org/D42895





More information about the cfe-commits mailing list