<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Jul 6, 2014 at 6:14 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class=""><br>
<div class="gmail_quote">On Sun, Jul 6, 2014 at 6:00 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">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?</blockquote>

</div><br></div>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....</div>

</div>
</blockquote></div><br></div><div class="gmail_extra">No tests fail if I comment out the call to WriteBlockInfoBlock. But the commit message from r125086 gives a hint: With these abbrevs, the output of `clang -cc1 -fms-extensions -emit-pch -o test.ast test.cc && llvm-bcanalyzer  test.ast` is nicer to read. So we should probably add the missing entries, and a test that FileChecks llvm-bcanalyzer output.</div>
</div>