<div dir="ltr">Are you looking at the lseek call in the raw_fd_ostream constructor? If so then I think I agree that the call might not be necessary (we'd have to look at all call sites to be sure though). But this patch is changing the call in the seek method.<div><br></div><div>Peter</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 8, 2016 at 8:53 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">It is not clear to me why we have a `lseek` call here in the first place. If this code is for "just in case if we are not at beginning of a file", I'd rather remove it.<div><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 8, 2016 at 8:43 PM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Looks like phab isn't sending mail, so mailing this the old fashioned way. This fixes "lld-link /linkrepro" if the reproducer file happens to be larger than 2GB.<div><br></div><div>Not sure if there's a good way to test this that doesn't involve creating huge files during check-llvm.<span class="m_-736610342577996034m_4924008811313688889m_-5779185822401621025HOEnZb"><font color="#888888"><br clear="all"><div><br></div>-- <br><div class="m_-736610342577996034m_4924008811313688889m_-5779185822401621025m_-2057198070215164425gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</font></span></div></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div>