<div dir="ltr">On Mon, Jul 15, 2013 at 8:37 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="im">> Is it possible for the buffer to only contain a single byte?<div><br></div></div><div>MemoryBuffer makes sure the file is null terminated, but the empty file is still a corner case where we have a single byte buffer over read.  I should fix it.</div>

<div><br></div><div>This is growing enough corner cases that I'm reconsidering my decision to not put this in llvm/lib/Support/ConvertUTF8Wrappers.cc.  I didn't like the idea of loading an entire response file (usually large) into a std::string, but the reality is that we're building a set of strings anyway.</div>

</div>
</blockquote></div><br></div><div class="gmail_extra">Considering that lld will need response file support, I should do this differently.</div></div>