[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