On 13 August 2017 at 15:44, Dibyendu Majumdar <mobile@xxxxxxxxxxxxxxx> wrote: > On 13 August 2017 at 15:18, Luc Van Oostenryck > <luc.vanoostenryck@xxxxxxxxx> wrote: >> On Sun, Aug 13, 2017 at 3:18 PM, Dibyendu Majumdar >> <mobile@xxxxxxxxxxxxxxx> wrote: >> >>>> Care to send your code, I can see where the problem could be? >>> >>> Yes apologies I should have mentioned the test case: >>> >>> https://github.com/dibyendumajumdar/dmr_c/blob/master/tests/lcc/cq.c >> >> No problem. >> >>> I am checking whether the slowdown is due to some other factor such as: >> >> I just gave a try and, I was a bit surprised as with the current master, >> here it takes me about 5s to get the output of test-linearize (which I find >> already a bit slow). It's far far from the time you have. >> >> OTOH with -rc4, it takes only 0.05s ! >> >> The situation is really weird, it's an interaction between the patches: >> - remove single-store shortcut (first slowdown to 0.8s) >> - fix ptrlist corruption while killing unreachable BBs (second slowdown to 2.9s) >> - mark pseudo users as deleted instead of removing them (third slowdown to 5.5s) >> >> All three patches were expected to create *some* slowdown, >> the first one very obviously and much more than the others. >> >> It's interesting to note that without the 'remove single shortcut' patch, the >> two others ones don't seem to have a significative effect on the time. >> > > Yes, reinstating the single store shortcut reduces time from 8.3 secs > to 0.16 seconds. > In the interest of getting the release out I recommend simply undoing the single store shortcut change. It was a last minute change and that is always risky. Regards Dibyendu -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
