<div dir="ltr">Hi,<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 29, 2014 at 9:09 PM, Jason Evans <span dir="ltr"><<a href="mailto:jasone@canonware.com" target="_blank">jasone@canonware.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On Jan 29, 2014, at 4:28 AM, İsmail Dönmez <<a href="mailto:ismail@donmez.ws">ismail@donmez.ws</a>> wrote:<br>


> I have 2 new failures:<br>
><br>
> thd_start:test/unit/prof_accum.c:83: Failed assertion: (bt_count_prev+(i-i_prev)) <= (bt_count) --> 6 > 1: thd_start<br>
<br>
</div>I'm guessing that this is due to the compiler being especially intelligent regarding mutual recursion for alloc_[01](), and I just added noinline attributes for those functions:<br>
<br>
        <a href="https://github.com/jemalloc/jemalloc/commit/526e4a59a2fe39e4f8bdf1ec0c0d2a5a557c3f62" target="_blank">https://github.com/jemalloc/jemalloc/commit/526e4a59a2fe39e4f8bdf1ec0c0d2a5a557c3f62</a><br>
<br>
However, if the compiler is being that smart, it may also be smart enough to do tail call optimization despite an attempt in the code to thwart optimization.  It appears that the gcc flag to disable this is -fno-optimize-sibling-calls, but I'm reluctant to resort to that unless the noinline attribute fails to do the job.<br>

</blockquote><div><br></div><div>This one is still failing, also adding <span style="font-family:arial,sans-serif;font-size:12.800000190734863px">-fno-optimize-sibling-calls to CFLAGS didn't fix it.</span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<div class="im"><br>
> [test_alignment_errors:test/integration/allocm.c:60: Failed assertion: (allocm(&p, &rsz, sz, (ffs(alignment)-1))) != (0) --> 0 == 0: test_alignment_errors<br>
<br>
</div>This is the equivalent failure to the mallocx failure you hit before.  Fixed:<br>
<br>
        <a href="https://github.com/jemalloc/jemalloc/commit/2850e90d0d42d0e2b54864949bfa41c59c3a8dc9" target="_blank">https://github.com/jemalloc/jemalloc/commit/2850e90d0d42d0e2b54864949bfa41c59c3a8dc9</a><br>
<br>
Testing is hard.  I am continually amazed by how much variation there is in compiler warnings and other behaviors, even between minor compiler revisions.  That said, most of the issues you hit are unique to 32-bit systems, so I really need to set up a 32-bit test system prior to the next release.<br>

</blockquote><div><br></div><div>This is working as expected :)</div><div><br></div><div>Regards.</div><div> </div></div></div></div>