[PATCH] D101777: [clang] p1099 1/5: [NFC] Break out BaseUsingDecl from UsingDecl

Nathan Sidwell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 24 08:25:28 PDT 2021


urnathan marked 2 inline comments as done.
urnathan added inline comments.


================
Comment at: clang/include/clang/AST/DeclCXX.h:3390
   ConstructorUsingShadowDecl(ASTContext &C, DeclContext *DC, SourceLocation Loc,
-                             UsingDecl *Using, NamedDecl *Target,
+                             BaseUsingDecl *Using, NamedDecl *Target,
                              bool TargetInVirtualBase)
----------------
davrec wrote:
> `BaseUsingDecl` here should remain `UsingDecl`.
> Additionally, I would add a `UsingDecl *getIntroducer() { return cast<UsingDecl>(UsingShadowDecl::getIntroducer()); }` method here.
Done, this required a further reordering of class definitions, so that UsingDecl is complete at this point.


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

https://reviews.llvm.org/D101777



More information about the cfe-commits mailing list