Fixed building on MSVC (#415)

Author: bkryza

src/common/clang/usr/llvm-19/USRGeneration.cpp

@@ -54,9 +54,13 @@ static bool printLoc(llvm::raw_ostream &OS, SourceLocation Loc,
 static StringRef GetExternalSourceContainer(const NamedDecl *D) {
   if (!D)
     return StringRef();
+
+#if !defined(_MSC_VER)
   if (auto *attr = D->getExternalSourceSymbolAttr()) {
     return attr->getDefinedIn();
   }
+#endif
+
   return StringRef();
 }
 
@@ -255,8 +259,12 @@ void USRGenerator::VisitFunctionDecl(const FunctionDecl *D) {
   D->getDeclName().print(Out, Policy);
 
   ASTContext &Ctx = *Context;
-  if ((!Ctx.getLangOpts().CPlusPlus || D->isExternC()) &&
-      !D->hasAttr<OverloadableAttr>())
+  if ((!Ctx.getLangOpts().CPlusPlus || D->isExternC())
+#if !defined(_MSC_VER)
+      &&
+      !D->hasAttr<OverloadableAttr>()
+#endif
+    )
     return;
 
   if (D->isFunctionTemplateSpecialization()) {
@@ -1215,6 +1223,7 @@ bool common::index::generateUSRForDecl(const Decl *D,
   // C++'s operator new function, can have invalid locations but it is fine to
   // create USRs that can identify them.
 
+#if !defined(_MSC_VER)
   // Check if the declaration has explicit external USR specified.
   auto *CD = D->getCanonicalDecl();
   if (auto *ExternalSymAttr = CD->getAttr<ExternalSourceSymbolAttr>()) {
@@ -1224,6 +1233,8 @@ bool common::index::generateUSRForDecl(const Decl *D,
       return false;
     }
   }
+#endif
+
   USRGenerator UG(&D->getASTContext(), Buf);
   UG.Visit(D);
   return UG.ignoreResults();

src/common/clang/usr/llvm-20/USRGeneration.cpp

@@ -58,9 +58,13 @@ static StringRef GetExternalSourceContainer(const NamedDecl *D)
 {
     if (!D)
         return StringRef();
+
+#if !defined(_MSC_VER)
     if (auto *attr = D->getExternalSourceSymbolAttr()) {
         return attr->getDefinedIn();
     }
+#endif
+
     return StringRef();
 }
 
@@ -273,8 +277,12 @@ void USRGenerator::VisitFunctionDecl(const FunctionDecl *D)
     Policy.SuppressTemplateArgsInCXXConstructors = true;
     D->getDeclName().print(Out, Policy);
 
-    if ((!LangOpts.CPlusPlus || D->isExternC()) &&
-        !D->hasAttr<OverloadableAttr>())
+    if ((!LangOpts.CPlusPlus || D->isExternC())
+#if !defined(_MSC_VER)
+        &&
+        !D->hasAttr<OverloadableAttr>()
+#endif
+)
         return;
 
     if (D->isFunctionTemplateSpecialization()) {
@@ -1362,6 +1370,7 @@ bool common::index::generateUSRForDecl(
     // C++'s operator new function, can have invalid locations but it is fine to
     // create USRs that can identify them.
 
+#if !defined(_MSC_VER)
     // Check if the declaration has explicit external USR specified.
     auto *CD = D->getCanonicalDecl();
     if (auto *ExternalSymAttr = CD->getAttr<ExternalSourceSymbolAttr>()) {
@@ -1371,6 +1380,8 @@ bool common::index::generateUSRForDecl(
             return false;
         }
     }
+#endif
+
     USRGenerator UG(&D->getASTContext(), Buf, LangOpts);
     UG.Visit(D);
     return UG.ignoreResults();