<div dir="ltr"><div>Looks good to me.  I suspect we can simplify callers of TryEmitDefinitionAsAlias in MS C++ ABI in followups.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 15, 2014 at 3:06 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Now with the patch :-(<br>
<div class="HOEnZb"><div class="h5"><br>
On 15 September 2014 18:06, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> Hi Reid,<br>
><br>
> The attached patch is rebased on top of trunk, implements the rules<br>
> described in <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62306#c6" target="_blank">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62306#c6</a> and<br>
> I think has all your previous suggestions.<br>
><br>
> I have sent an email to cxx-abi-dev documenting the above rules, but<br>
> got no replies so far.<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></div>