<div dir="ltr">Looks good, but I'd let Duncan comment.<div><br></div><div>There is an instance of "Aliassee" in clang-3.patch that should probably be made consistent with the rest.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, May 14, 2014 at 12:08 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">
Starting a new thread to get a fresh view of the state of the patches.<br>
<br>
The introduction of GlobalObject was already committed. These patches are<br>
* rebased on top of trunk.<br>
* include the feedback from the previous code review. Let me know if I<br>
missed anything.<br>
* Splits the manual removal of getAliasedGlobal to a separate patch.<br>
* Includes some linker fixes I found while rereading the previous version.<br>
<br>
One thing that I noticed with the discussion about the C api is that<br>
LLVMAddAlias should probably not be changed, even if the constructor<br>
of GlobalAlias is changed. I will try to split this into an<br>
independent patch.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>