rsmith added a comment. The direction here seems fine to me.
================ Comment at: clang/include/clang/Sema/Sema.h:2478 - Decl *ActOnDeclarator(Scope *S, Declarator &D); + Decl *ActOnDeclarator(Scope *S, Declarator &D, bool NextTokIsEqual = false); ---------------- Can we set a `HasInitializer` flag on `Declarator` instead (much like we provide a `FunctionDefinitionKind`)? This flag seems a bit ad-hoc (and also doesn't cover C++ braced initializers as named). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90871/new/ https://reviews.llvm.org/D90871 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits