<div>A couple of things:</div>* AttributedStmt::CreateEmpty isn't allocating enough space for the Attr*s.<br>* Please capitalize the parameters to the AttributedStmt member functions.<br><br>LGTM with those fixed.<br><br>
<div class="gmail_quote">On Fri, Jul 6, 2012 at 3:24 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Did anyone have time to look at this?<div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 5, 2012 at 9:42 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<div><br></div><div>This patch changes AttrVec to inline storage of attributes in AttributedStmt. I also had to add a template parameter for container type to specific_attr_iterator to make it handle ArrayRef along with AttrVec. Updated serialization code, but there's still no good way to test it.</div>


<div><br></div><div>Please take a look at this patch.</div><span><font color="#888888"><div><div><br></div>-- <br>
<div>Regards,</div><div>Alexander Kornienko</div></div>
</font></span></blockquote></div><br>
</div>
</div></div></blockquote></div><br>