<div dir="ltr">Hi Nick and Bill,<div><br></div><div>Attached is a new version of the patch that checks against cur_buffer_size. This works with the cases I've been using but again I haven't run the official tests on this change.</div>
<div><br></div><div>Thanks,</div><div>Joseph Kain</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 5, 2013 at 9:46 AM, Joseph Kain <span dir="ltr"><<a href="mailto:joekain@gmail.com" target="_blank">joekain@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Nick,<div><br></div><div>Thanks for looking over my patch.  I'd be happy to add the compare against cur_buffer_size.  I'll try to write up such a change today and then send a new patch to the list.</div>

<div><br></div><div>Joseph Kain</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Thu, Sep 5, 2013 at 1:37 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Bill, could you take a look at this? I don't have a checkout of compiler-rt that builds.<br>
<br>
It seems obviously-correct to me, but I wish it did a compare against cur_buffer_size to make sure it's in range.<br>
<br>
Nick<br>
<br>
Joseph Kain wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
Hi,<br>
<br>
I've been seeing a crash in llvm_gcda_emit_arcs() on x86_64.  The crash<br>
occurs executing a movdqa instruction with an unaligned src address.<br>
  The attached patch to the compiler-rt project fixes the problem by<br>
using memcpy() to read data from the write_buffer[] in GCDAProfiling.c.<br>
<br>
This is my first patch submission to llvm so please let me know if I've<br>
missed any steps.  I'm not on the mailing list so please include me on<br>
any replies.<br>
<br>
I haven't run the tests on this change but would be happy to do so if<br>
someone could point me to documentation on how to do so.  I couldn't<br>
find testing instructions linked from<br>
<a href="http://llvm.org/docs/DeveloperPolicy.html" target="_blank">http://llvm.org/docs/<u></u>DeveloperPolicy.html</a>.  I have built the change and<br>
run it with the programs I was testing originally.  With this change the<br>
crash no longer occurs.<br>
<br>
Thanks,<br>
Joseph Kain<br>
<br>
<br></div></div>
______________________________<u></u>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</blockquote>
<br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br>Joseph Kain
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Joseph Kain
</div>