AlliedModders - New JIT in SourceMod 1.6

AlliedModders (https://forums.alliedmods.net/index.php)

- General (https://forums.alliedmods.net/forumdisplay.php?f=58)

- - New JIT in SourceMod 1.6 (https://forums.alliedmods.net/showthread.php?t=222967)

New JIT in SourceMod 1.6

Hi all,

I've just checked in a rewrite of the SourcePawn JIT (bug 5827), the piece that is responsible for running plugin code. It is mostly to reduce the complexity of the JIT so I can play with new optimizations and not be totally confused about what's going on.

It should work exactly the same as before, except in 3,800 fewer lines of code. However, if you're running SourceMod 1.6 nightlies and get new crashes, please let me know.

Re: New JIT in SourceMod 1.6

Cool stuff! I though SourcePawn2 is actually new SP which will be implemented later.
I will test it ASAP and report any found issues. I hope it will be implemented in upcoming SM 1.5.0, because it sounds cool :bee:

Re: New JIT in SourceMod 1.6

Well I downloaded 1.6.0-hg3980.zip for Windows and immediately got an error in a plugin which is using this function:

PHP Code:


		
			
GetRealClientCount(bool:inGameOnly)
{
    new clients[MaxClients], numClients;

    for (new i = 1; i <= MaxClients; i++)
    {
        if (((inGameOnly) ? IsClientInGame(i) : IsClientConnected(i)) && !IsFakeClient(i))
        {
            clients[numClients++] = i;
        }
    }

    return numClients;
}

An error:

Code:

[SM] Plugin encountered error 17: Stack memory leaked by native

[SM] Displaying call stack trace for plugin "myplugin"

Reported on line "new clients[MaxClients], numClients;"
Previously on older versions I havent had this error. And I foudn a solution: when I change it to clients[MAXPLAYERS + 1] the error is not appearing. Should I file a bug instead?

Re: New JIT in SourceMod 1.6

Root - Thanks for testing. Could you send me this plugin? Your code is fine so there is a bug in the JIT.