sammccall added inline comments.

================
Comment at: clang-tools-extra/clangd/Selection.cpp:615
+      if (isa<CXXConstructorDecl>(D))
+        return SourceRange();
+      // This will capture Field, Function, MSProperty, NonTypeTemplateParm and
----------------
we may have to blacklist deduction guides too?


================
Comment at: clang-tools-extra/clangd/unittests/SelectionTests.cpp:378
           "ObjCPropertyRefExpr"},
+      {"struct foo { [[int has^h<:32:>]]; };", "FieldDecl"},
+      // FIXME: This should be class itself instead.
----------------
can you add the other special cases we discussed offline? conversion operations 
should be triggered by `operator`, destructors by `~`.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75106



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

Reply via email to