A couple of new trim-related branches

Carl Worth cworth at cworth.org
Fri Feb 15 19:07:57 PST 2013


I've just pushed two new branches to the apitrace repository, which I'd
like to propose for merging. They are

  1. trim-more-no-side-effects

	This is a branch to address the bug raised in issue #112. With
	this branch in place, both "trim --prune" and "trim --deps
	--trim-spec=no-side-effects" are much more effective than before
	at eliminating calls without side effects.

	There are a few, related bug fixes in the branch as well.

	I've also pushed a corresponding branch to apitrace-tests called
	trim-testing-improvements with new tests to better exercise this
	functionality.

  2. trim-optimization

	This is a hugely important performance optimization for
	"apitrace trim". I've had this around for a while, but I can't
	remember if I submitted it here on the list earlier.

	I'm actually embarrassed for anyone who has been trying to use
	"apitrace trim --auto" without this patch, since it's so
	painfully slow. (Even with the patch, things are still slower
	than they should be, but the biggest offender,
	(std::set<unsigned>::insert), should drop off of the profile).

	For people that have been getting apitrace from my personal
	repository, I just noticed that this performance optimization
	has been missing from the master branch that I've had there
	since January, (though it had been present there last
	year). Sorry about that.

In addition to these branches, I've also pushed a "cworth" branch that
octopus-merges all of my proposed branches, (the above two, plus
replay-rename, and mixed-program-program-object). That's the same branch
I've pushed to my personal repository's "master".

José, if the "cworth" branch looks like clutter to you, I can remove
it. Otherwise, I'll be happy to just rebase/force-push it as relevant
branches get merged into master. Just let me know what you'd like to see
here.

Thanks,

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/apitrace/attachments/20130215/568ee5d3/attachment.pgp>


More information about the apitrace mailing list