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.


On Tue, Sep 3, 2013 at 4:19 PM, Eli Friedman <[email protected]> wrote:

> Patch attached.  I think this is mostly straightforward, but I'l like to
> get this reviewed by someone more familiar with the serialization code.
>
> Fixes PR16635.
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to