<div dir="ltr">There's no padding both in NativeReferenceIvarsV1 and in NativeReferenceIvarsV2, so __attribute((packed)) would be no-op?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 18, 2013 at 1:51 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I was mentioning similiar things to this for structs.<br>
<br>
struct foo {<br>
char c;<br>
int x;<br>
} __attribute__((packed));<div class="HOEnZb"><div class="h5"><br>
<br>
On 11/18/2013 3:46 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Nov 18, 2013 at 1:38 PM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>><u></u>wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Should we use attribute packed too. I am not sure if this will raise<br>
compatibility concerns across all the machines that build lld.<br>
<br>
</blockquote>
Which attributes are you referring to?<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks<br>
<br>
Shankar Easwaran<br>
<br>
<br>
On 11/18/2013 3:05 PM, <a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    Beside compatibility with existing files (which we don't have yet),<br>
there is another reason for V1 vs V2: size.  The NativeReferenceIvar is<br>
twice the size (8 bytes vs 4 bytes).  There will be a lot of these in<br>
files.  We should have both structs.  When a native object file is written,<br>
the Writer should see if everything is small enough that just<br>
NativeReferenceIvarsV1 structs can be used.  If not, it uses<br>
NativeReferenceIvarsV2. The Reader should always handle both.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2217" target="_blank">http://llvm-reviews.chandlerc.<u></u>com/D2217</a><br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
<br>
<br>
</blockquote>
--<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted<br>
by the Linux Foundation<br>
<br>
<br>
</blockquote></blockquote>
<br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</div></div></blockquote></div><br></div>