Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Call ffi.typeof() directly #212

Closed
AltraMayor opened this issue Mar 14, 2019 · 1 comment
Closed

Call ffi.typeof() directly #212

AltraMayor opened this issue Mar 14, 2019 · 1 comment
Labels
enhancement good first issue
Milestone
Version 1.1

Comments

@AltraMayor
Copy link
Owner

luaL_get_ctypeid(), which GT blocks call multiple times while interacting with the Lua policy (see #210 for a reference), gets a reference to ffi.typeof() running the Lua code return require('ffi').typeof. Most of this overhead goes away if someone finds out how to call the C function of LuaJIT under ffi.typeof().
@AltraMayor AltraMayor added this to the Advanced features milestone Mar 14, 2019
@AltraMayor
Copy link
Owner Author

ffi.typeof() is available in dependencies/luajit-2.0/src/lib_ffi.c, search for LJLIB_CF(ffi_typeof).

AltraMayor added a commit that referenced this issue Mar 22, 2023
@AltraMayor
lua: reduce garbage production at luaL_get_ctypeid()
d0dd2b0 
This commit closes #212
AltraMayor added a commit that referenced this issue Mar 22, 2023
@AltraMayor
lua: reduce garbage production at luaL_get_ctypeid()
aa07afe 
This commit closes #212
AltraMayor added a commit that referenced this issue Mar 24, 2023
@AltraMayor
lua: reduce garbage production at luaL_get_ctypeid()
d34e071 
This commit closes #212
AltraMayor added a commit that referenced this issue Mar 24, 2023
@AltraMayor
lua: reduce garbage production at luaL_get_ctypeid()
46a6f61 
This commit closes #212
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue
Projects
None yet
Milestone
Version 1.1
Development

No branches or pull requests
1 participant
@AltraMayor