<div dir="ltr">If you can separate --gdb-index as a separate patch, please do.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Sep 10, 2016 at 1:26 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D24267#538684" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24267#538684</a>, @ruiu wrote:<br>
<br>
> I'm still reading this patch and haven't got the whole picture yet. A few swing-by comments. Please do it incrementally next time. You don't need to make up a complete, working code from day one -- but you can incrementally build a feature in the public repository, and it is actually more appreciated way of doing it than dropping a complete large patch out of the blue.<br>
<br>
<br>
</span>I am really sorry, do not want inconvenience for anyone. My suggestion is next then. What if I`ll split this patch to several ? I mean them will not be workable, first one can just generate the gdb_index section itself and some part of data. So it can be a sequence of patches which will make the functionality workable after the last one is committed.<br>
What do you think ?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24267" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24267</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>