in reply to Re^3: Autovivifying XS routine
in thread Autovivifying XS routine

It will be interesting to see if Want can detect/supply the calling context more quickly than the overloading. The latter is notoriously slow...

The way I understand things, overload stuff is called from the code that interprets the de-referencing opcodes, individually in all five(?) cases. If it's slow, that's probably not because it has to determine the kind of reference needed, it should know. Not that it cared anyway.

...c&p the logic directly into your own XS code I suppose?

That is the plan.

Anno

Replies are listed 'Best First'.
Re^5: Autovivifying XS routine
by BrowserUk (Patriarch) on Jun 01, 2007 at 16:53 UTC
    ... overload stuff is called from the code that interprets the de-referencing opcodes ... If it's slow, that's probably not because it has to determine the kind of reference needed,

    Exactly. It knows what it's got to do. It only needs to chase a few pointers in magic to find the callback--and still it is notoriously slow.

    A cursory peek at want.xs shows an aweful lot of logic, including several physically and logically nested loops tracing opcode trees. I won't pretend to have understood half of what I saw in there, but it looks expensive, which supports my findings from a while ago.

    My point is that it might be faster, and simpler, to have gimme_a_ref() return a ref blessed into a package that overloads the dereference operators. Especially as Alter is already using (as far as I understood your other thread), two levels of inside-out-style dereferencing which is already slow.

    Additionally, what happens if someone assigns the return from gimme_a_ref() to a scalar, and then attempts to dereference it later? You'll have no applicable 'current context' at the time of the call from which to determine what use will be made of what you return.

    Admittedly that can be addressed through documentation, but you'll be denying flexibility. You'll also create the situation of requiring the additional overhead of determining the calling context for every use.


    Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
    "Science is about questioning the status quo. Questioning authority".
    In the absence of evidence, opinion is indistinguishable from prejudice.
      ...peek at want.xs shows an aweful lot of logic

      Right. I disentangled the part that I'd need and ended up with (almost) half the code from Want.xs. Speed issues aside, that's not reasonable.

      what happens if someone assigns the return from gimme_a_ref() to a scalar, and then attempts to dereference it later?

      That would be no different from other autovivifying situations. You can say

      my $x; $x->{hihi}->{haha} = 'hoho';
      but you can't keep $x->{hihi} for later, as in
      my $x; my $y = $x->{hihi}; $y->{haha} = 'hoho';
      So I'm not worried much about that effect, but the autovivifying option is out for the reason above.

      Alter is already using ..., two levels of inside-out-style dereferencing

      It's not quite as bad. Alter::ego is one hash access slower than the inside-out equivalent Hash::Util::refaddr. That's the difference that matters afaik, otherwise they do about the same amount of almost nothing.

      Anno