<div dir="ltr">lgtm</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 5, 2014 at 4:25 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">Currently unnamed_addr is in a strange state. It is stored in<br>
GlobalValue, the language reference talks about "unnamed_addr aliases"<br>
but the verifier rejects them.<br>
<br>
We should move it to GlobalObject or make it legal in aliases.<br>
<br>
Given the recent changes, It seems natural to allow them in aliases:<br>
<br>
* It is a property of how it is accessed, not of the data itself.<br>
* It is perfectly possible to write code that depends on the address<br>
of an alias.<br>
<br>
The attached patch then makes unname_addr legal for aliases. One side<br>
effect is that the syntax changes for a corner case: In globals,<br>
unnamed_addr is now printed before the address space.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>