aaron.ballman added inline comments.

================
Comment at: clang/tools/libclang/CXCursor.cpp:1279-1281
+  if (const UsingType *Using = Ty->getAs<UsingType>())
+    if (const UsingShadowDecl *Shadow = Using->getFoundDecl())
+      if (const auto *TD = dyn_cast_or_null<TypeDecl>(Shadow->getTargetDecl()))
----------------
Hmm, we're exposing `CXType_Using` but here we're not giving back a cursor to 
the `UsingType` but instead looking through that type to get down to a 
declaration and are returning that.

Should we split these in two different cursors?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140075

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

Reply via email to