[PATCH] D90871: [Sema] Fold VLAs to constant arrays in a few more contexts

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 15 10:17:49 PST 2020


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



More information about the cfe-commits mailing list