<div dir="ltr">So it is a vague concern which is not really an actionable thing or something. Sorry for writing such comment, but I'd think that expressing what I feel while I'm reading a patch could still be a useful feedback. I'm not trying to block this or other patches -- this looks good to me.<div><br></div><div>As to the size of the linker, it is for sure important, particularly when you want to hack it.<div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 25, 2016 at 3:15 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think part of what we're getting out of this is understanding.  If we wait until we need it for the linker specifically, then we don't have any experience so we're starting from scratch, and the past few weeks will be an extra few weeks since we have to do all this anyway.<br><br>We also need it for not just the linker.  Lldb will need it eventually, and the dumper needs it no matter what for testing.  <br><br>Even if we have to make some changes to accommodate LLD's needs, at least we will be starting from something and have a deep knowledge of the format, ultimately saving us some time.<br><br>You mention the code size of LLD being small.  Is the size of the linker important?  I thought we are mostly interested in optimizing the speed<div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 25, 2016 at 3:05 PM Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
At a high level, it looks good to me, but as I probably mentioned earlier, I have a little concern that we are having an API for PDB that is too complete at this moment. The COFF linker is 7000 lines long in total, and if we keep this pace, we'll probably end up having a PDB library which is larger than the linker. Even if we'd eventually need it, I'm more comfortable if we do as-needed basis.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D22693" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22693</a><br>
<br>
<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br></div></div></div>