Skip to content

Conversation

@wenyongh
Copy link
Collaborator

@wenyongh wenyongh commented Aug 10, 2022

Thread data should not be destroyed when thread exits, or other thread
may not be able to join it. This PR saves the thread data into thread data
list when thread exits, sets thread status and stores the return value, so
that other thread can join it.

Also set MEM_TOP_DOWN flag for Windows VirtualAlloc to yield LLVM
JIT relocation error.
And set opt/size level to 3 for LLVM JIT for future use, currently the flags
are not used by LLVM JIT.

@wenyongh wenyongh changed the title Fix windows thread data issue Fix windows thread data issue and enhance os_mmap Aug 10, 2022
@wenyongh wenyongh changed the title Fix windows thread data issue and enhance os_mmap Fix windows thread data issue and enhance windows os_mmap Aug 10, 2022
@wenyongh wenyongh merged commit e23acfa into bytecodealliance:main Aug 11, 2022
@wenyongh wenyongh deleted the fix_win_issues branch August 15, 2022 07:50
loganek pushed a commit to loganek/wasm-micro-runtime that referenced this pull request Aug 31, 2022
…liance#1372)

Thread data should not be destroyed when thread exits, or other thread
may not be able to join it. This PR saves the thread data into thread data
list when thread exits, sets thread status and stores the return value, so
that other thread can join it.

Also set MEM_TOP_DOWN flag for Windows VirtualAlloc to yield LLVM
JIT relocation error.

And set opt/size level to 3 for LLVM JIT for future use, currently the flags
are not used by LLVM JIT.
vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
…liance#1372)

Thread data should not be destroyed when thread exits, or other thread
may not be able to join it. This PR saves the thread data into thread data
list when thread exits, sets thread status and stores the return value, so
that other thread can join it.

Also set MEM_TOP_DOWN flag for Windows VirtualAlloc to yield LLVM
JIT relocation error.

And set opt/size level to 3 for LLVM JIT for future use, currently the flags
are not used by LLVM JIT.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant