<div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word;"><div><div class="im">

<div></div></div><div>It would be much easier to handle such additions as separate patches once the main patch has gone in. Big patches take much longer to review. We haven't quite dealt with all of the issues there yet.</div>

</div></div></blockquote><div><br>I realized that after I re-posted it. I can try to break it down into more digestible pieces.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div style="word-wrap: break-word;"><div>Also, friends are in a state of flux, so it's not worth trying to (de-)serialize them now. Once the AST settles down and access control is turned on by default, it will make sense to implement PCH support with friends.</div>

</div></blockquote><div><br>Sounds good. I'll try to take out the friend stuff.  <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div style="word-wrap: break-word;"><div><div class="im"><blockquote type="cite"><div class="gmail_quote"><div></div></div>The DefinitionData isn't attached but the previous declaration is set correctly. IIRC, the same situation happens with injected class names.<br>

</blockquote></div></div></div></blockquote><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word;"><div><div class="im">

<blockquote type="cite"></blockquote>Since you posted the original patch, a better solution has come up. When we start a definition of a CXXRecordDecl, we go back and fix up all of the DefinitionData pointers.</div></div>

</div></blockquote><div><br>That is a better solution :) <br></div><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word;">

<div>Only the 2.7 branch of Clang/LLVM is in code freeze. The trunk is in its usual frenzy of activity.</div><div><br></div><div>Overall, this patch isn't quite ready to go in. <br></div></div></blockquote><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div style="word-wrap: break-word;"><div><br></div><div>I think fixing conversion functions and the ctor-initializers are crucial for this patch to go in.<br></div></div></blockquote><div><br>Sounds good. I'll take another turn on it. Unfortunately, I'm pretty busy for the next week or so. Maybe extending  PCH support for clang (expressions, too) would make a good GSoC project :)<br>

</div></div><br>Andrew Sutton<br><a href="mailto:andrew.n.sutton@gmail.com">andrew.n.sutton@gmail.com</a><br>