<div dir="rtl"><div dir="ltr">Sure, committed in r224159.</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2014-12-12 23:33 GMT+02:00 Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It'd be great if someone could commit this for me, I don't have commit access.<br>
<br>
Thanks,<br>
- Kim<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Dec 12, 2014 at 10:01 PM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> I'm not sure how to test this either, but the bug is obvious and your patch<br>
> LGTM.<br>
><br>
><br>
> 2014-12-12 22:36 GMT+02:00 Kim Gräsman <<a href="mailto:kim.grasman@gmail.com">kim.grasman@gmail.com</a>>:<br>
>><br>
>> Hello,<br>
>><br>
>> I stumbled over a bug in MemoryBufferMMapFile, where it fails to<br>
>> report errors from its constructor.<br>
>><br>
>> Attached patch changes the EC argument of its constructor to be passed<br>
>> by-ref, which matches what its client (getOpenFileImpl) expects.<br>
>><br>
>> I've experimented with test cases to reproduce this, but I haven't<br>
>> found any clear-cut ways to do it (it's hidden behind too many<br>
>> incidental details). I've attached my last attempt, but it closes the<br>
>> global stdin fd as a side-effect, so it's no good.<br>
>><br>
>> For what it's worth,<br>
>> - Kim<br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
><br>
</div></div></blockquote></div></div>