<div dir="ltr">How could you fix it with V1? Relocation targets are stored in NCS_TargetsTable chunk, but the index (targetIndex in NativeReferenceIvarsV1) pointing to an item in the chunk has only 16 bits. That's the limitation of NativeReferenceIvarsV1 format. It seems that we have no choice other than using a data structure having bigger integer for the taget table indices.<div>

<br></div><div>As to addend, please look at the definition of NativeReferenceIvarsV2. "addend" field is defined as int64_t, which is big enough to directly store addend. So we should eliminate the indirection for addend for saving disk space, memory and for runtime efficiency.</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 18, 2013 at 10:29 AM, Shankar Kalpathi Easwaran <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  This should be fixed with V1. I dont think there is a need for V2 at this time. I couldnot follow the changes behind the addendsTableIndex ? is this part of a different cleanup ?<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2217" target="_blank">http://llvm-reviews.chandlerc.com/D2217</a><br>
</blockquote></div><br></div>