Nikodemus Siivola <nikodemus@...> writes:
> The attached patch makes CLOS go faster when MOP is afoot: in current
> CVS HEAD all ways of accessing slots with the exception of
> STANDARD-INSTANCE-ACCESS slow down by a factor of ~17 when there are
> user defined methods on SLOT-VALUE-USING-CLASS or its compatriots.
> With my changes the difference is goes down to a slowdown by a factor
> of ~4 when using MOPified accesses.
> Patch attached for your delectation -- builds and passes tests, but
> has not yet been heavily tested otherwise.
> A simplistic benchmark is also attached.
I think I once deleted something a bit like this -- it might not have
been quite so comprehensive, and certainly I didn't measure the speed
impact on MOP code, but the motivation for deleting it was to save
space. Three functions per effective slot definition is really quite a
lot; maybe measure the space used with something like gsharp / climacs?
Having said that, I just looked at your patch and it doesn't look like
it's introducing any extra stuff -- in which case, good job :-) Maybe I
didn't delete these things after all.