[PATCH] D101554: [clangd] Add support for the `defaultLibrary` semantic token modifier

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 3 01:41:55 PDT 2021


kadircet added a comment.

LG from my side.



================
Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:470
+  if (auto *OT = dyn_cast<ObjCObjectType>(T))
+    return scopeModifier(OT->getInterface());
   return llvm::None;
----------------
is this guaranteed to be non-null ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101554



More information about the cfe-commits mailing list