[PATCH] D124915: Check for resource exhaustion when recursively parsing declarators

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 10:08:44 PDT 2022


aaron.ballman marked an inline comment as done.
aaron.ballman added inline comments.


================
Comment at: clang/include/clang/Parse/Parser.h:802
+  /// more in that case. Use this in code that may recurse deeply (for example,
+  /// in template instantiation) to avoid stack overflow.
+  void runWithSufficientStackSpace(SourceLocation Loc,
----------------
erichkeane wrote:
> Hmm... The Parser version of this might be used for Template Instantiation?  Should we remove the SEMA one and move all uses to the Parser then?
Nope, but I could be less silly and just... use the one from Sema... instead of moving it to Parser. Derp!


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

https://reviews.llvm.org/D124915



More information about the cfe-commits mailing list