r212425 - Add an AST node for __leave statements, hook it up.

Chandler Carruth chandlerc at google.com
Sun Jul 6 18:14:13 PDT 2014


On Sun, Jul 6, 2014 at 6:00 PM, Nico Weber <thakis at chromium.org> wrote:

> I didn't modify AddStmtExprs() in ASTWriter.cpp, since none of the SEH
> nodes are in there (and e.g. STMT_CXX_TRY isn't either). Does anyone know
> what this function is good for, and if it should be updated with STMTs that
> are currently missing from there?


It should be used as part of the AST serialization, specifically to build
up the types of expressions which can be serialized... Not sure why we're
missing so much from that, or why it is factored this way. Most of the
statement seralization is nicely factored into ASTWriterStmt.cpp but for
some reason this vestige remains. Maybe it is completely dead code now? We
may just not need the meta-records it is streaming....
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140706/00544a16/attachment.html>


More information about the cfe-commits mailing list