search for: cur_buffer_size

Hi Nick, 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. Joseph Kain On Thu, Sep 5, 2013 at 1:37 AM, Nick Lewycky <nicholas at mxc.ca> wrote: > Bill, could you take a look at this? I don't have a checkout of > compiler-rt that builds. > > It se...

Hi Nick and Bill, 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. Thanks, Joseph Kain On Thu, Sep 5, 2013 at 9:46 AM, Joseph Kain <joekain at gmail.com> wrote: > Hi Nick, > > Thanks for looking over my patch. I'd be happy to add t...

Bill, could you take a look at this? I don't have a checkout of compiler-rt that builds. It seems obviously-correct to me, but I wish it did a compare against cur_buffer_size to make sure it's in range. Nick Joseph Kain wrote: > Hi, > > I've been seeing a crash in llvm_gcda_emit_arcs() on x86_64. The crash > occurs executing a movdqa instruction with an unaligned src address. > The attached patch to the compiler-rt project fixes the problem b...

Hi, I've been seeing a crash in llvm_gcda_emit_arcs() on x86_64. The crash occurs executing a movdqa instruction with an unaligned src address. The attached patch to the compiler-rt project fixes the problem by using memcpy() to read data from the write_buffer[] in GCDAProfiling.c. This is my first patch submission to llvm so please let me know if I've missed any steps. I'm not on