<div dir="ltr">Seems completely reasonable to me. I have a slight preference for the test to check that we actually emit the definition of SPXTrace (there are various ways we could still get this completely wrong and pass this test anyway), but LGTM either way.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 3, 2013 at 4:19 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com" target="_blank">eli.friedman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Patch attached.  I think this is mostly straightforward, but I'l like to get this reviewed by someone more familiar with the serialization code.<div>
<br></div><div>Fixes PR16635.</div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>