<div dir="ltr"><div class="gmail_default" style>Awesome, thanks guys!</div><div class="gmail_default" style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 11, 2013 at 11:34 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Committed in r172214.<br>
<span class="HOEnZb"><font color="#888888"><br>
Eli<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Jan 11, 2013 at 8:30 AM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>> wrote:<br>
> oops, I svn diff'ed this from the unit test directory.  Re-submitting.<br>
><br>
><br>
> On Fri, Jan 11, 2013 at 11:16 AM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>
>><br>
>> On Fri, Jan 11, 2013 at 6:01 AM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>><br>
>> wrote:<br>
>> > Thanks, Jim.<br>
>> ><br>
>> > I should make it clear that I have no commit privs - can you commit<br>
>> > this?<br>
>> ><br>
>><br>
>> I will commit this for you a bit later today, Michael.<br>
>><br>
>> Eli<br>
>><br>
>><br>
>><br>
>><br>
>> ><br>
>> > On Thu, Jan 10, 2013 at 4:05 PM, Jim Grosbach <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Thanks! This is good to commit.<br>
>> >><br>
>> >> The main style guide is <a href="http://llvm.org/docs/CodingStandards.html" target="_blank">http://llvm.org/docs/CodingStandards.html</a>. When<br>
>> >> things change, migration is a slow process when there's a change, as we<br>
>> >> don't generally do a en-masse update of existing code, but rather<br>
>> >> update<br>
>> >> things a bit at a time. I think the naming conventions for variables,<br>
>> >> functions, etc, changed sometime in the last year or so. Something like<br>
>> >> that, anyway.<br>
>> >><br>
>> >> -Jim<br>
>> >><br>
>> >> On Jan 10, 2013, at 1:00 PM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>> wrote:<br>
>> >><br>
>> >><br>
>> >> Done. New patch attached.<br>
>> >><br>
>> >> (I'll update the patch in the bug later)<br>
>> >><br>
>> >> Was there a recent style normalization effort in the codebase?  (I try<br>
>> >> to<br>
>> >> emulate the style of the context)<br>
>> >><br>
>> >><br>
>> >> On Thu, Jan 10, 2013 at 3:53 PM, Jim Grosbach <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Missed one.<br>
>> >>><br>
>> >>> s/CreateMemoryManager/createMemoryManager/<br>
>> >>><br>
>> >>> OK to commit with that change.<br>
>> >>><br>
>> >>><br>
>> >>> On Jan 10, 2013, at 11:50 AM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>><br>
>> >>> wrote:<br>
>> >>><br>
>> >>> Ping?<br>
>> >>><br>
>> >>><br>
>> >>> On Tue, Jan 8, 2013 at 6:03 PM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>><br>
>> >>> wrote:<br>
>> >>>><br>
>> >>>><br>
>> >>>><br>
>> >>>><br>
>> >>>> On Tue, Jan 8, 2013 at 5:52 PM, Jim Grosbach <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>><br>
>> >>>> wrote:<br>
>> >>>>><br>
>> >>>>> Looks good to me, with a couple nits:<br>
>> >>>>><br>
>> >>>>> +  virtual RecordingJITMemoryManager* CreateMemoryManager() {<br>
>> >>>>><br>
>> >>>>> The '*' should bind to the identifier, not the type. The function<br>
>> >>>>> name<br>
>> >>>>> should start with lower case. There's a few other instances in the<br>
>> >>>>> patch of<br>
>> >>>>> the former. Please double-check.<br>
>> >>>><br>
>> >>>><br>
>> >>>> All fixed.<br>
>> >>>><br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> Thanks!<br>
>> >>>>><br>
>> >>>>> -Jim<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> On Jan 8, 2013, at 2:43 PM, Michael Muller <<a href="mailto:mmuller@google.com">mmuller@google.com</a>><br>
>> >>>>> wrote:<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> New patch attached, incorporates Misha's comments and current trunk.<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> On Fri, Jan 4, 2013 at 7:44 PM, Misha Brukman <<a href="mailto:brukman@gmail.com">brukman@gmail.com</a>><br>
>> >>>>> wrote:<br>
>> >>>>>><br>
>> >>>>>> On Wed, Dec 19, 2012 at 1:55 PM, Michael Muller<br>
>> >>>>>> <<a href="mailto:mmuller@enduden.com">mmuller@enduden.com</a>><br>
>> >>>>>> wrote:<br>
>> >>>>>>><br>
>> >>>>>>> See <a href="http://llvm.org/bugs/show_bug.cgi?id=13678" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=13678</a> for more history.<br>
>> >>>>>>><br>
>> >>>>>>> See also:<br>
>> >>>>>>><br>
>> >>>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-August/052766.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-August/052766.html</a><br>
>> >>>>>>><br>
>> >>>>>>><br>
>> >>>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120910/150422.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120910/150422.html</a><br>

>> >>>>>>><br>
>> >>>>>>><br>
>> >>>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120917/150801.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120917/150801.html</a><br>

>> >>>>>>><br>
>> >>>>>>><br>
>> >>>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-December/056829.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-December/056829.html</a><br>
>> >>>>>>><br>
>> >>>>>>> When you run out of space writing to a buffer, the buffer<br>
>> >>>>>>> management<br>
>> >>>>>>> code<br>
>> >>>>>>> simply stops writing at the end of the buffer.  It is the<br>
>> >>>>>>> responsibility of<br>
>> >>>>>>> the caller to verify that it has stayed in bounds and perform a<br>
>> >>>>>>> retry<br>
>> >>>>>>> with<br>
>> >>>>>>> a larger memory estimate if not.  The function writing code does<br>
>> >>>>>>> this, but<br>
>> >>>>>>> the exception table code following it does not.  The end result is<br>
>> >>>>>>> that<br>
>> >>>>>>> exception table pointers can get registered pointing to invalid<br>
>> >>>>>>> data,<br>
>> >>>>>>> causing<br>
>> >>>>>>> seg-faults when an exception is thrown.<br>
>> >>>>>><br>
>> >>>>>><br>
>> >>>>>> Minor drive-by style comments (I wish there were a web UI for<br>
>> >>>>>> entering<br>
>> >>>>>> these in context):<br>
>> >>>>>><br>
>> >>>>>> s/;;/;/<br>
>> >>>>>> ASSERT_TRUE(actual == expected) --> ASSERT_EQ(expected, actual)<br>
>> >>>>>> add space around binary operators (-, *, etc.), e.g.:<br>
>> >>>>>><br>
>> >>>>>> +      ActualSize = (CurBufferPtr-BufferBegin)*2;<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> I'm sure I copied that from somewhere, but looking at the file now I<br>
>> >>>>> see nothing resembling such formatting.  Fixed.<br>
>> >>>>><br>
>> >>>>>><br>
>> >>>>>> fix arg alignment of "ActualSize" here:<br>
>> >>>>>><br>
>> >>>>>> +    while (true) {<br>
>> >>>>>> +      BufferBegin = CurBufferPtr =<br>
>> >>>>>> MemMgr->startExceptionTable(F.getFunction(),<br>
>> >>>>>> +<br>
>> >>>>>> ActualSize);<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> Done.<br>
>> >>>>><br>
>> >>>>>><br>
>> >>>>>> Misha<br>
>> >>>>><br>
>> >>>>><br>
>> >>>>><br>
>> >>>>><br>
>> >>>>> --<br>
>> >>>>> Michael Muller<br>
>> >>>>> Ym9yZWQ/<br>
>> >>>>> <ExceptionTableOverflowFix-v5.patch><br>
>> >>>>><br>
>> >>>>><br>
>> >>>><br>
>> >>>><br>
>> >>>><br>
>> >>>> --<br>
>> >>>> Michael Muller<br>
>> >>>> Ym9yZWQ/<br>
>> >>><br>
>> >>><br>
>> >>><br>
>> >>><br>
>> >>> --<br>
>> >>> Michael Muller<br>
>> >>> Ym9yZWQ/<br>
>> >>><br>
>> >>><br>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> Michael Muller<br>
>> >> Ym9yZWQ/<br>
>> >> <ExceptionTableOverflowFix-v7.patch><br>
>> >><br>
>> >><br>
>> ><br>
>> ><br>
>> ><br>
>> > --<br>
>> > Michael Muller<br>
>> > Ym9yZWQ/<br>
>> ><br>
>> > _______________________________________________<br>
>> > llvm-commits mailing list<br>
>> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>> ><br>
><br>
><br>
><br>
><br>
> --<br>
> Michael Muller<br>
> Ym9yZWQ/<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Michael Muller<br>Ym9yZWQ/
</div>