[PATCH] D99231: [AST] Add introspection support for more base nodes

[Nathan James via Phabricator via cfe-commits]

njames93 added subscribers: aaron.ballman, rsmith.
njames93 added inline comments.


================
Comment at: clang/include/clang/AST/DeclCXX.h:2270
   SourceLocation getEllipsisLoc() const {
-    assert(isPackExpansion() && "Initializer is not a pack expansion");
+    if (!isPackExpansion())
+      return {};
----------------
steveire wrote:
> njames93 wrote:
> > I'm not sure about this change, but I'm guessing there's not a nice way to predicate these kinds of things
> Yep.
@rsmith @aaron.ballman Any issues with changing the contract of these functions to not assert for the purpose of this feature?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99231