SoulTch wrote:

@Sirraide @cor3ntin Thanks for the review.

I addressed the requested changes:
- fixed the PR name
- refactored to reduce duplicated code including `ParseLexedObjCMethodDefs` 
(`Parser::ParseFunctionBody`)
- added a release note

During the refactoring, I made a couple of changes that differ from the current 
behavior - I don't think they harm the original intent.
 - `UnmarkAsLateParsedTemplate` is now called regardless of whether the 
function body is valid.
 - The template depth assertions now run regardless of whether the function 
body is valid.

Please let me know if there is anything else that would be good to fix!

https://github.com/llvm/llvm-project/pull/218168
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to