Repository navigation
__builtin_setjmp/__builtin_longjmp crashes when mrb_load_exec #5950
Description
Target platform: Windows 10 (21H2)
Toolchain: llvm-mingw-20230130-ucrt-x86_64
When I execute mrb_load_exec and it throws an exception, it crashes.
I noticed that the information here says that we should use the standard C library functions declared in <setjmp.h> in user code instead of the builtins.
So I tried editing the <throw.h>:
#if defined(__APPLE__) || defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__)
#define MRB_SETJMP _setjmp
#define MRB_LONGJMP _longjmp
// #elif defined(__MINGW64__) && defined(__GNUC__) && __GNUC__ >= 4
// #define MRB_SETJMP __builtin_setjmp
// #define MRB_LONGJMP __builtin_longjmp
#else
#define MRB_SETJMP setjmp
#define MRB_LONGJMP longjmp
#endifThis time there was no crash. I hope this message is helpful~
Activity
Revert "mruby/throw.h: do not use __builtin_setjmp for GCC; close #5950"
dearblue commentedon Jan 7, 2024
It was fixed once, but then later reverted.
4fc69ce
Is it ok if this issue is not reopened?
matz commentedon Jan 7, 2024
This change made
mrbtestto fail often. It was unacceptable for us.If @LanzaSchneider reopens the issue, it's OK. But without further information, we can do little.
mruby/throw.h: do not use __builtin_setjmp for GCC; close mruby#5950
throw.h: use setjmp/longjmp with clang on MinGW