<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 13, 2014 at 10:31 AM, 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"><div class="">> It was nice that `Alignment` was packed in here before.<br>
><br>
> I wonder if it's worth the complexity of storing `Alignment` in<br>
> `GlobalValue`, even though it's only valid for `GlobalObject`?  Maybe<br>
> using the `Value::SubclassData` model?  I'm not sure how much this<br>
> bloat matters (1M globals => 8 MB).<br>
<br>
</div>Probably. This revised patch fixes the style issue and uses the parent<br>
padding bits.<br>
<br>
The sizes I get with the patch on x86_64 are:<br>
<br>
Constant:       56<br>
GlobalValue:    64<br>
GlobalObject:   88<br>
GlobalVariable: 112<br>
Function:       152<br>
<br>
Without the patch they are<br>
<br>
Constant:       56<br>
GlobalValue:    88<br>
GlobalObject:   X<br>
GlobalVariable: 112<br>
Function:       152<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div><div class="gmail_extra">The first patch looks good, I'm just opening the second patch.</div><div class="gmail_extra"><br>I think previously you suggested splitting this into GlobalName / GlobalValue instead of GlobalValue / GlobalObject.  I kind of liked GlobalName.  Any reason to split this way instead?  Does it avoid excessive API churn?</div>
</div>