From b91bc93fd33f2158745b37c9e091c6fa4d1a4fd5 Mon Sep 17 00:00:00 2001 From: Roberto Ierusalimschy Date: Wed, 9 Sep 2015 10:44:07 -0300 Subject: 'setobj2t' incorporated into 'luaV_fastset' + 'invalidateTMcache' is not needed in the fast track (as it does not create new entries) --- lvm.h | 32 +++++++++++++++++++++----------- 1 file changed, 21 insertions(+), 11 deletions(-) (limited to 'lvm.h') diff --git a/lvm.h b/lvm.h index ccdacd4a..b65061ac 100644 --- a/lvm.h +++ b/lvm.h @@ -1,5 +1,5 @@ /* -** $Id: lvm.h,v 2.37 2015/08/03 19:50:49 roberto Exp roberto $ +** $Id: lvm.h,v 2.38 2015/08/03 20:40:26 roberto Exp roberto $ ** Lua virtual machine ** See Copyright Notice in lua.h */ @@ -70,17 +70,27 @@ else luaV_finishget(L,t,k,v,aux); } -#define luaV_fastset(L,t,k,aux,f,v) \ +/* +** Fast track for set table. If 't' is a table and 't[k]' is not nil, +** call GC barrier, do a raw 't[k]=v', and return true; otherwise, +** return false with 'slot' equal to NULL (if 't' is not a table) or +** 'nil'. (This is needed by 'luaV_finishget'.) Note that, if the macro +** returns true, there is no need to 'invalidateTMcache', because the +** call is not creating a new entry. +*/ +#define luaV_fastset(L,t,k,slot,f,v) \ (!ttistable(t) \ - ? (aux = NULL, 0) \ - : (aux = f(hvalue(t), k), \ - ttisnil(aux) ? 0 \ - : (luaC_barrierback(L, hvalue(t), v), 1))) - -#define luaV_settable(L,t,k,v) { const TValue *aux; \ - if (luaV_fastset(L,t,k,aux,luaH_get,v)) \ - { invalidateTMcache(hvalue(t)); setobj2t(L, cast(TValue *,aux), v); } \ - else luaV_finishset(L,t,k,v,aux); } + ? (slot = NULL, 0) \ + : (slot = f(hvalue(t), k), \ + ttisnil(slot) ? 0 \ + : (luaC_barrierback(L, hvalue(t), v), \ + setobj2t(L, cast(TValue *,slot), v), \ + 1))) + + +#define luaV_settable(L,t,k,v) { const TValue *slot; \ + if (!luaV_fastset(L,t,k,slot,luaH_get,v)) \ + luaV_finishset(L,t,k,v,slot); } -- cgit v1.2.1