[xdebug-general] Re: xdebug performance enhancements

From: Taavi Burns <taavi.burns[@]gmail.com>
Date: Tue, 5 Jul 2011 17:18:19 -0400

Will do on the -O2.

I pushed a couple of fixes to the cache_filename branch that you might want, too (they automatically showed up in the pull request). One saves a pointer to the local, just-created filename; the other puts the cache data in xdebug_globals, for thread safety. I also changed it to be a xdebug_coverage_file*. :)


On 2011-07-05, at 5:05 PM, Derick Rethans <derick[@]xdebug.org> wrote:
> Hi!
> On Sun, 3 Jul 2011, Taavi Burns wrote:
>> On Sun, Jul 3, 2011 at 7:50 AM, Derick Rethans <derick[@]xdebug.org> wrote:
>>> I've no idea either yet. I guess either writing a mail like this or
>>> using the "Create pull request" functionality works. I'll be looking at
>>> your patches later today. Thanks so far!
>> Sounds great. I opened 2 pull requests for you. I hear github makes it
>> _really_ easy to incorporate the changes if they're good as-is (and
>> allows for commenting etc). :)
> I've merged them (but not yet pushed them, will do some testing first). 
> I had to change the cache filename one because you used void * and not 
> xdebug_coverage_file* and some cosmetic changes.
>> When I was poking around at the xdebug build itself (trying to get C
>> profiling working), I noticed that xdebug itself is compiled with -O0.
>> Is there a particular reason for that? I tried forcing it to the usual
>> -O2, and it had a similar percentage speedup as the two optimizations
>> I just submitted (and works well with my changes, too, for even more
>> speed).
> The reason was that some (old) compilers caused issues with -O2 and so I 
> forced -O0. I don't think that's required anymore so feel free to make a 
> feature branch for that too unless I beat you to it.
> cheers,
> Derick
> -- 
> Like Xdebug? Consider a donation: http://xdebug.org/donate.php
> Xdebug | http://xdebug.org | xdebug-general@lists.xdebug.org
> twitter: @derickr and @xdebug
Received on Tue Jul 05 2011 - 22:18:25 BST

This archive was generated by hypermail 2.2.0 : Mon Jun 25 2018 - 06:00:04 BST