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

Reply via email to