Phase 2F: API Table Access Helpers (lapi.cpp) - Static Function Elimination

Converted 3 static table access helper functions to lua_State private methods:
- auxgetstr() → lua_State::auxGetStr()
- auxsetstr() → lua_State::auxSetStr()
- getGlobalTable() → lua_State::getGlobalTable()

**Changes**:
- Added 3 private method declarations to lua_State class (lstate.h)
- Added 5 friend function declarations for public API functions
- Converted static functions to member methods (lapi.cpp)
- Updated 6 call sites across 5 API functions

**Function Details**:
- auxgetstr() → auxGetStr(const TValue *t, const char *k): Gets value from
  table using string key, handles fast path and metamethod fallback
- auxsetstr() → auxSetStr(const TValue *t, const char *k): Sets value in
  table using string key, handles fast path and metamethod fallback
- getGlobalTable() → getGlobalTable(TValue *gt): Retrieves global table
  from registry[LUA_RIDX_GLOBALS]

**Friend Functions** (need access to table access helpers):
- lua_getglobal() - gets global variable by name
- lua_getfield() - gets table field by string key
- lua_setglobal() - sets global variable by name
- lua_setfield() - sets table field by string key
- lua_load() - loads chunk and sets up global table as first upvalue

**Call Sites Updated**: 6 locations across 5 API functions
- lua_getglobal: calls getGlobalTable() and auxGetStr()
- lua_getfield: calls auxGetStr()
- lua_setglobal: calls getGlobalTable() and auxSetStr()
- lua_setfield: calls auxSetStr()
- lua_load: calls getGlobalTable() to setup function environment

**Impact**: Table access helpers properly encapsulated. These functions handle
the common pattern of string-keyed table access with fast path optimization.

**Files Changed**: 2 files (lstate.h, lapi.cpp)

**Testing**: All tests pass ✅ (2.11s single run)
**Benchmark**: 5-run average = 2.12s ✅ (50% faster than 4.20s baseline!)
**Risk**: MEDIUM (core API functions, multiple call sites, but well-tested)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: Peter Neiss

src/core/lapi.cpp

@@ -589,17 +589,17 @@ LUA_API int lua_pushthread (lua_State *L) {
 */
 
 
-static int auxgetstr (lua_State *L, const TValue *t, const char *k) {
-  TString* const str = TString::create(L, k);
-  LuaT tag = L->getVM().fastget(t, str, s2v(L->getTop().p), [](Table* tbl, TString* strkey, TValue* res) { return tbl->getStr(strkey, res); });
+int lua_State::auxGetStr(const TValue *t, const char *k) {
+  TString* const str = TString::create(this, k);
+  LuaT tag = getVM().fastget(t, str, s2v(getTop().p), [](Table* tbl, TString* strkey, TValue* res) { return tbl->getStr(strkey, res); });
   if (!tagisempty(tag))
-    api_incr_top(L);
+    api_incr_top(this);
   else {
-    setsvalue2s(L, L->getTop().p, str);
-    api_incr_top(L);
-    tag = L->getVM().finishGet(t, s2v(L->getTop().p - 1), L->getTop().p - 1, tag);
+    setsvalue2s(this, getTop().p, str);
+    api_incr_top(this);
+    tag = getVM().finishGet(t, s2v(getTop().p - 1), getTop().p - 1, tag);
   }
-  lua_unlock(L);
+  lua_unlock(this);
   return novariant(tag);
 }
 
@@ -609,19 +609,19 @@ static int auxgetstr (lua_State *L, const TValue *t, const char *k) {
 ** table; so, an emergency collection while using the global table
 ** cannot collect it.
 */
-static void getGlobalTable (lua_State *L, TValue *gt) {
-  Table* const registry = hvalue(G(L)->getRegistry());
+void lua_State::getGlobalTable(TValue *gt) {
+  Table* const registry = hvalue(G(this)->getRegistry());
   const LuaT tag = registry->getInt(LUA_RIDX_GLOBALS, gt);
   (void)tag;  /* avoid not-used warnings when checks are off */
-  api_check(L, novariant(tag) == LUA_TTABLE, "global table must exist");
+  api_check(this, novariant(tag) == LUA_TTABLE, "global table must exist");
 }
 
 
 LUA_API int lua_getglobal (lua_State *L, const char *name) {
   TValue gt;
   lua_lock(L);
-  getGlobalTable(L, &gt);
-  return auxgetstr(L, &gt, name);
+  L->getGlobalTable(&gt);
+  return L->auxGetStr(&gt, name);
 }
 
 
@@ -639,7 +639,7 @@ LUA_API int lua_gettable (lua_State *L, int idx) {
 
 LUA_API int lua_getfield (lua_State *L, int idx, const char *k) {
   lua_lock(L);
-  return auxgetstr(L, L->getStackSubsystem().indexToValue(L,idx), k);
+  return L->auxGetStr(L->getStackSubsystem().indexToValue(L,idx), k);
 }
 
 
@@ -761,29 +761,29 @@ LUA_API int lua_getiuservalue (lua_State *L, int idx, int n) {
 /*
 ** t[k] = value at the top of the stack (where 'k' is a string)
 */
-static void auxsetstr (lua_State *L, const TValue *t, const char *k) {
-  TString* const str = TString::create(L, k);
-  api_checkpop(L, 1);
-  const int hres = L->getVM().fastset(t, str, s2v(L->getTop().p - 1), [](Table* tbl, TString* strkey, TValue* val) { return tbl->psetStr(strkey, val); });
+void lua_State::auxSetStr(const TValue *t, const char *k) {
+  TString* const str = TString::create(this, k);
+  api_checkpop(this, 1);
+  const int hres = getVM().fastset(t, str, s2v(getTop().p - 1), [](Table* tbl, TString* strkey, TValue* val) { return tbl->psetStr(strkey, val); });
   if (hres == HOK) {
-    L->getVM().finishfastset(t, s2v(L->getTop().p - 1));
-    L->getStackSubsystem().pop();  /* pop value */
+    getVM().finishfastset(t, s2v(getTop().p - 1));
+    getStackSubsystem().pop();  /* pop value */
   }
   else {
-    setsvalue2s(L, L->getTop().p, str);  /* push 'str' (to make it a TValue) */
-    api_incr_top(L);
-    L->getVM().finishSet(t, s2v(L->getTop().p - 1), s2v(L->getTop().p - 2), hres);
-    L->getStackSubsystem().popN(2);  /* pop value and key */
+    setsvalue2s(this, getTop().p, str);  /* push 'str' (to make it a TValue) */
+    api_incr_top(this);
+    getVM().finishSet(t, s2v(getTop().p - 1), s2v(getTop().p - 2), hres);
+    getStackSubsystem().popN(2);  /* pop value and key */
   }
-  lua_unlock(L);  /* lock done by caller */
+  lua_unlock(this);  /* lock done by caller */
 }
 
 
 LUA_API void lua_setglobal (lua_State *L, const char *name) {
   TValue gt;
-  lua_lock(L);  /* unlock done in 'auxsetstr' */
-  getGlobalTable(L, &gt);
-  auxsetstr(L, &gt, name);
+  lua_lock(L);  /* unlock done in 'auxSetStr' */
+  L->getGlobalTable(&gt);
+  L->auxSetStr(&gt, name);
 }
 
 
@@ -802,8 +802,8 @@ LUA_API void lua_settable (lua_State *L, int idx) {
 
 
 LUA_API void lua_setfield (lua_State *L, int idx, const char *k) {
-  lua_lock(L);  /* unlock done in 'auxsetstr' */
-  auxsetstr(L, L->getStackSubsystem().indexToValue(L,idx), k);
+  lua_lock(L);  /* unlock done in 'auxSetStr' */
+  L->auxSetStr(L->getStackSubsystem().indexToValue(L,idx), k);
 }
 
 
@@ -1012,7 +1012,7 @@ LUA_API int lua_load (lua_State *L, lua_Reader reader, void *data,
     if (f->getNumUpvalues() >= 1) {  /* does it have an upvalue? */
       /* get global table from registry */
       TValue gt;
-      getGlobalTable(L, &gt);
+      L->getGlobalTable(&gt);
       /* set global table as 1st upvalue of 'f' (may be LUA_ENV) */
       *f->getUpval(0)->getVP() = gt;
       luaC_barrier(L, f->getUpval(0), &gt);

src/core/lstate.h

@@ -760,6 +760,18 @@ struct lua_State : public GCBase<lua_State> {
 
   // Phase 2E: Friend function that needs access to API helper
   friend void lua_rotate(lua_State *L, int idx, int n);
+
+  // Phase 2F: API table access helpers (lapi.cpp)
+  int auxGetStr(const TValue *t, const char *k);
+  void auxSetStr(const TValue *t, const char *k);
+  void getGlobalTable(TValue *gt);
+
+  // Phase 2F: Friend functions that need access to table access helpers
+  friend int lua_getglobal(lua_State *L, const char *name);
+  friend int lua_getfield(lua_State *L, int idx, const char *k);
+  friend void lua_setglobal(lua_State *L, const char *name);
+  friend void lua_setfield(lua_State *L, int idx, const char *k);
+  friend int lua_load(lua_State *L, lua_Reader reader, void *data, const char *chunkname, const char *mode);
 };