<br><br><div class="gmail_quote">On Wed, Nov 19, 2008 at 12:13 PM, Gary Chambers <span dir="ltr"><<a href="mailto:gazzaguru2@btinternet.com">gazzaguru2@btinternet.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Perhaps the compiler could ask the class (or meta) of the method being compiled as to whether to inline a selector. All after the fact until a full recompile of the class, of course...</blockquote><div><br></div><div>Doesn't really help. One wants the property to be of the receiver, not of the sending context. Inheritance gets in the way also since a subclass that overrides the setting would need to reimplement all methods that send class to get its own setting, and even then super sends could invoke inherited code compiled with the wrong setting. Alas this idea doesn't work.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
<br>
Regards, Gary.<br>
<br>
----- Original Message ----- From: "Igor Stasenko" <<a href="mailto:siguctua@gmail.com" target="_blank">siguctua@gmail.com</a>><br>
To: "The general-purpose Squeak developers list" <<a href="mailto:squeak-dev@lists.squeakfoundation.org" target="_blank">squeak-dev@lists.squeakfoundation.org</a>><br>
Sent: Wednesday, November 19, 2008 8:03 PM<br>
Subject: Re: [squeak-dev] Lightweight Classes<div><div></div><div class="Wj3C7c"><br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2008/11/19 Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
On Sun, Nov 16, 2008 at 7:11 PM, Igor Stasenko <<a href="mailto:siguctua@gmail.com" target="_blank">siguctua@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2008/11/17 Hernán Morales Durand <<a href="mailto:hernan.morales@gmail.com" target="_blank">hernan.morales@gmail.com</a>>:<br>
> Dear all,<br>
> In the attachment you will find an implementation of Lightweight<br>
> Classes<br>
> for Squeak that follows the original paper "Debugging Objects" of > Hinkle<br>
> et<br>
> al. However I was unable to set up the #class method in the<br>
> LightweightClass<br>
> to answer the real class of the lightwighted object, is this possible<br>
> currently in Squeak?.<br>
><br>
> What I did:<br>
> -Created the LightweightClass as in the old VisualWorks > implementation.<br>
> The<br>
> only difference here was CompiledMethod's cannot be copied, so I used<br>
> #clone.<br>
> -Created and installed an UnstoredMethod (a CompiledMethod subclass) > in<br>
> the<br>
> LightweightClass's methodDictionary. Since I want to store the source<br>
> code<br>
> but not through the changes log, I borrowed the MethodWrappers idea of<br>
> storing the state (sourceCode) in a class instance variable, and > compile<br>
> without logging. The methods I borrowed from MW are:<br>
><br>
> -#copyWithTrailerBytes: - I think the superimplementor in > CompiledMethod<br>
> could use "self class" to avoid reimplementors.<br>
> -#tweakClassFormat - which set the format but I don't know why and<br>
> cannot<br>
> find documentation on this.<br>
> -#objectAt: I suspected the class literal was stored in the literal<br>
> frame<br>
> everytime I accepted a method, but for the simple test below still<br>
> answer<br>
> the LightweightClass and not the real class.<br>
><br>
> Here is the test:<br>
><br>
> | aDate |<br>
> aDate := Date today.<br>
> aDate becomeLightweight.<br>
> aDate dispatchingClass compile: 'day ^43' notifying: nil.<br>
> aDate day. " 43 (works) "<br>
> aDate perform: #class. " Date ------> (works) "<br>
> aDate class. " {Date} ------> LightweightClass (wrong) "<br>
><br>
<br>
The only cause of this can be compiler.<br>
Instead of generating an instruction for sending #class message, it<br>
generates a bytecode to fetch the class from receiver oop , instead of<br>
sending real message.<br>
</blockquote>
<br>
IMO the bug is in the VM. The compiler generates bytecode 199, the special<br>
selector class send. But it is the VM that decides to short-cut this send<br>
and implement it by accessing the receiver's class directly instead of<br>
actually sending the message. I doubt very much this short-circuiting has<br>
any impact on performance (VisualWorks has never inlined this special<br>
selector send), and it is very easy to fix in the VM. A number of other<br>
special selector bytecodes become ordinary sends (e.g. next nextPut: etc).<br>
If class were important for performance one could implement something<br>
analogous to the at cache that speeds up at:, at:put: and size for special<br>
selectors. The special selector bytecode would check that the receiver of<br>
the class message had the standard implementation (primitive in Object) and<br>
answer the class directly. But this isn't going to save significant time.<br>
To fix this change<br>
bytecodePrimClass<br>
| rcvr |<br>
rcvr := self internalStackTop.<br>
self internalPop: 1 thenPush: (self fetchClassOf: rcvr).<br>
self fetchNextBytecode.<br>
to<br>
bytecodePrimClass<br>
messageSelector := self specialSelector: 23.<br>
argumentCount := 0.<br>
self normalSend.<br>
<br>
</blockquote>
<br>
Agreed, it seem a VM feature, kind of :)<br>
As temporary workaround, one could modify compiler to generate a<br>
normal send bytecode instead of using a short bytecode for special<br>
selector which #class is. I'm not sure how easy/hard such modification<br>
could be made.<br>
<br>
And i agree, that such optimization buys a little. I didn't seen a<br>
code which does a heavy numerical crunching, where speed is essential,<br>
and at same time needs a #class sends. To my sense a #class appears in<br>
places where we need to deal with polymorphism or doing something on<br>
meta levels - but in such areas, a correctess is far more important<br>
than speed.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> Any hint would be appreciated.<br>
> Best regards.<br>
><br>
> Hernán<br>
><br>
> PS: Just to avoid duplicate efforts, I wrote a LightweightClasses<br>
> Browser<br>
> which will be available as soon as I find a solution to the problem<br>
> above.<br>
><br>
<br>
--<br>
Best regards,<br>
Igor Stasenko AKA sig.<br>
<br>
</blockquote></blockquote>
<br>
<br>
-- <br>
Best regards,<br>
Igor Stasenko AKA sig.<br>
<br>
</blockquote>
<br>
<br></div></div>
--------------------------------------------------------------------------------<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
</blockquote>
<br>
<br>
</blockquote></div><br>