ykjit / yklua

yk-enabled Lua interpreter
4 stars 4 forks source link

Add cstack test #45

Closed Pavel-Durov closed 1 year ago

Pavel-Durov commented 1 year ago

Add cstack test - it runs 1000 times on my machine

Pavel-Durov commented 1 year ago

bors try

bors[bot] commented 1 year ago

try

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here. For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

vext01 commented 1 year ago

bors r+

bors[bot] commented 1 year ago

Build failed:

vext01 commented 1 year ago

It failed on CI, so I guess we close this?

ltratt commented 1 year ago

Before we kill it, we should briefly discuss why it failed (did it fail because of other yk bugs or ...?).

Pavel-Durov commented 1 year ago

Looks like it failed on other tests - coroutine.lua:

21:10:17 + echo ### YKD_SERIALISE_COMPILATION=0 coroutine.lua ###
21:10:17 + YKD_SERIALISE_COMPILATION=0 ../src/lua -e_U=true coroutine.lua
21:10:17 ### YKD_SERIALISE_COMPILATION=0 coroutine.lua ###
21:10:17 testing coroutines
21:10:17 to-be-closed variables in coroutines
21:10:17 testing 'coroutine.close'
21:10:17 
21:10:17  >>> testC not active: skipping coroutine API tests <<<
21:10:17 
21:10:17 lua: /ci/yk/ykllvm/llvm/lib/IR/Value.cpp:1101: void llvm::ValueHandleBase::RemoveFromUseList(): Assertion `*PrevPtr == this && "List invariant broken"' failed.
Pavel-Durov commented 1 year ago

Removed coroutine test :point_right: de6710c50ab6162f98a97fd7f4f4a7e0d27a81f3 It fails on my machine as well

ltratt commented 1 year ago

Please squash.

Pavel-Durov commented 1 year ago

squashed 👉 2b36a8d8963d1a891bfc926b3b2308442e472e7c

ltratt commented 1 year ago

bors r+

bors[bot] commented 1 year ago

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here. For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.