[PATCH] Fix Remote MCJIT protocol

Renato Golin renato.golin at linaro.org
Thu Jan 9 12:56:09 PST 2014


On 9 January 2014 19:29, Alp Toker <alp at nuanti.com> wrote:

> Drive-by comment: Can you move all the rc asserts into a single check
> within ReadBytes()?
>
> They seem to check the same thing -- that the requested number of bytes
> was actually read.


Right, good point. Will do that. I didn't want to change much in
ChildTarget, since it was going to die, but I can make some amends, even if
it gets killed in the process later on.

--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140109/b494e9ea/attachment.html>


More information about the llvm-commits mailing list