<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div class="im">This patch looks great to me.  I committed it in r103301.</div>

</div></blockquote><div><br></div><div>Great!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div>I'm also interested in working on C++ PCH support in the next couple days.  What can I work on that won't conflict with your work?  If possible, I'd like to shoot for getting C++ PCH support working in the next week or two.</div>

</div></blockquote><div><br></div><div>The remainder of the patch that Doug reviewed dealt with classes, constructors, destructors, and conversion operators needed a little work (to be loaded lazily I think). The next patch will definitely include read/write support for CXXBaseSpecifier and CXXBaseOrMemberInitializer. I cropped those out of this patch, but I don't think that there wasn't anything wrong with them.</div>

<div><br></div><div>I'll try to get the next patch out Monday.</div><div><br></div><div>I don't plan on touching the serialization of templates and template parameters The last time I tried to do something with it, I kept getting assertions that I shouldn't serialize dependent types or expressions. That was probably in December, though. Things may have changed since then :)</div>

<div><br></div><div>I haven't looked at any of the CXX expressions either.</div></div><br>Andrew Sutton<br><a href="mailto:andrew.n.sutton@gmail.com">andrew.n.sutton@gmail.com</a><br>