[PATCH] D90568: [clang] Add [is|set]Nested methods to NamespaceDecl

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 10 14:51:57 PST 2022


shafik added inline comments.


================
Comment at: clang/lib/AST/ASTContext.cpp:8767-8770
         const_cast<ASTContext &>(*Context), Context->getTranslationUnitDecl(),
         /*Inline*/ false, SourceLocation(), SourceLocation(),
         &Context->Idents.get("std"),
+        /*PrevDecl*/ nullptr, /*Nested*/ false);
----------------
Overriding my old comment from a while ago,


================
Comment at: clang/lib/AST/ASTContext.cpp:8957-8959
+                               /*Inline*/ false, SourceLocation(),
                                SourceLocation(), &Context->Idents.get("std"),
+                               /*PrevDecl*/ nullptr, /*Nested*/ false);
----------------



================
Comment at: clang/lib/AST/ItaniumMangle.cpp:605-607
         /*Inline*/ false, SourceLocation(), SourceLocation(),
         &getASTContext().Idents.get("std"),
+        /*PrevDecl*/ nullptr, /*Nested*/ false);
----------------



================
Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:386
+      AST, AST.getTranslationUnitDecl(), false, SourceLocation(),
+      SourceLocation(), &HLSL, PrevDecl, false);
   HLSLNamespace->setImplicit(true);
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90568



More information about the cfe-commits mailing list