[PATCH] Fix Remote MCJIT protocol
Renato Golin
renato.golin at linaro.org
Tue Jan 14 14:50:58 PST 2014
Thanks Andrew!
Committed with the change in r199261. Let's see if the ARM tests will keep green. ;)
--renato
================
Comment at: tools/lli/ChildTarget/ChildTarget.cpp:241
@@ +240,3 @@
+ // Write the result.
+ rc = WriteBytes(&Status, 4);
+ assert(rc == 4);
----------------
Andy Kaylor wrote:
> It doesn't look to me like the size of LLIMessageStatus is guaranteed to be 4. Maybe you can specify an explicit type for the enum.
Good point, I've changed it to uint32_t.
http://llvm-reviews.chandlerc.com/D2527
More information about the llvm-commits
mailing list