RE: lua 5.1.1 patch for bitwise operators

["Grellier, Thierry" <t-grellier@...>]

No the patch for direct threaded was enabling it or not depending on
environments. I forgot to add jumptable this is why when activated it
doesn't compile. It was probably activated in your environment, so it
may be worth using it.
I've attached a revised version of the patch where is doesn't try to be
clever and have direct threaded disabled by default. When disabled it
has absolutely no overhead. You can decide to turn it on according to
your needs. Then it will increase size for sure and it may improve
performances.
It is recommended using -fno-crossjumping (accordingly to your gcc
version) for lvm.o if you want to use it.

The size numbers I gave were without direct threaded patch enabled and
purely reflect the overhead of the bitwise operators.

-----Original Message-----
From: lua-bounces@bazar2.conectiva.com.br
[mailto:lua-bounces@bazar2.conectiva.com.br] On Behalf Of Doug Rogers
Sent: Tuesday, June 20, 2006 7:46 PM
To: Lua list
Subject: Re: lua 5.1.1 patch for bitwise operators

Grellier, Thierry wrote:

> Well, it also includes the patch for direct threaded code in virtual
> machine for gcc, which isn't enabled on x86, and can be discarded in
> luaconf.h

I like the patch. Seems to work for me, though I would prefer to have
the patch without the direct threaded code. It failed to compile for me
and I had to go in manually to remove it. I do not know what else in the
patch is related to the direct threaded code, so it is possible that
your sizes show more than just the bitwise operators.

> [lua-5.1.1 ]                       % size src/lua
>    text    data     bss     dec     hex filename
>  114770     644      16  115430   1c2e6 src/lua
> [lua-5.1.1.patched ]               % size src/lua
>    text    data     bss     dec     hex filename
>  117330     644      16  117990   1cce6 src/lua

Doug

-- 
--__-__-____------_--_-_-_-___-___-____-_--_-___--____
Doug Rogers - ICI - V:703.893.2007x220 www.innocon.com
-_-_--_------____-_-_-___-_--___-_-___-_-_---_--_-__-_

Attachment: bitwise_directthreaded.patch.gz
Description: bitwise_directthreaded.patch.gz