<br><div class="gmail_extra">On Tue, Nov 6, 2012 at 9:17 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">>> ><br>
>><br>
>> A slightly expanded version of this will make for a great comment in<br>
>> the code :-) Seriously, please document the purpose of the new<br>
>> relocMap data structure you're adding.<br>
>><br>
><br>
> Uh hey, good point. Done :)<br>
<br>
</div>Great. At the moment I get a mental Parse Error when reading the<br>
comment (especially the first sentence), though. I'll try to find time<br>
to study the code better a bit later, maybe it will make the error go<br>
away.<br>
<div><div class="h5"><br></div></div></blockquote><div><br></div><div>I'll reread it a bit and see if I can make it more clear.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
</div></div>Hmm, I don't know if it makes sense to flatten out all info about<br>
every section like this. DWARF relocations can appear for a number of<br>
sections and other things may be added in the future. However, it's<br>
probably not worth over-thinking it at this point as it may be<br>
refactored later if the need arises.</blockquote><div><br></div><div>Agreed, I've got about 4 more sections for just the accelerator tables and</div><div>then fission will add a few more. I'm totally up for ways to refactor or</div>
<div>redesign all of this to make sense. I played with it for a couple days and</div><div>let my desire to avoid allocating 3GB of RAM just to dump a file take</div><div>priority. That said, I'd be up for anything reasonable that feels better.</div>
<div><br></div><div>Thanks for the comments!</div><div><br></div><div>-eric </div></div><br></div>