<div dir="ltr"><div>[Moving LLVM to BCC and adding lldb-dev]<br></div><div><br></div>Hi Piotr!<div><br></div><div><div>Thanks for the note.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">
On Wed, Feb 19, 2014 at 2:54 PM, Piotr Rak <span dir="ltr"><<a href="mailto:piotr.rak@gmail.com" target="_blank">piotr.rak@gmail.com</a>></span> wrote:<br><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div>But when we are requested to do read at boundary of two such not merged VMRanges, we will read correct data from from file to the end of 'last_entry->data.GetRangeEnd()' and then we fill rest with '\0'.</div>

<div>I think that we should split our requested read operation, to all such ranges in this case.</div><div><br></div><div>Is that correct?</div><div><br></div></div></blockquote><div><br></div><div>For the elf core file support, each of those memory regions represent a memory segment of the elf file.  I don't *think* you should be having any memory objects which span across segments.  So while it might be true that a request for a contiguous range of VM memory starts off in one segment's VM map entry and might zero fill if it would cover multiple segments that were not merged, I'm not sure what would be driving the need to do that.</div>
<div><br></div><div>What is the scenario where the need to read memory over multiple segments is showing up?</div><div><br></div><div>If we did need to do it, I think you are right in that we'd need to merge reads from each of the VM regions that overlapped with the read request.</div>
<div><br></div><div>Let me know what you find!</div><div><br></div><div>Thanks,</div><div>Todd <br></div></div><div>-- <br></div><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'">
<tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px">
 Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34)">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div></div></div>