-
Notifications
You must be signed in to change notification settings - Fork 667
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.
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
Move spec test to the last commit of Apr.3 #3293
Move spec test to the last commit of Apr.3 #3293
Conversation
f44e6e5
to
b8a5c1f
Compare
540350b
to
0cb2e72
Compare
7e34e4e
to
f7adb29
Compare
# multi-module only enable in interp mode and aot mode | ||
if [[ ${ENABLE_MULTI_MODULE} -eq 1 ]]; then | ||
if [[ "${RUNNING_MODE}" != "classic-interp" \ | ||
&& "${RUNNING_MODE}" != "fast-interp" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should also add aot, && "${RUNNING_MODE}" != "aot"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aot + multi-modules will fail the linking test case. Tend to fix it in another PR.
90476ef
to
1290c0d
Compare
- update spec test to commit bc76fd79cfe61033d7f4ad4a7e8fc4f996dc5ba8 on Apr. 3 - upddate wabt binary to 1.0.34 to support newer spec cases - Add comparision between table declared elem type and elem segment value type - Add a function to decide whether to execute test cases in a running mode - keep using interpreter in GC spec because wat2wasm in wabt can't compile if.wast w/o errors - re-factoring threads spec test case processing - since wabt 1.0.34 release isn't compatible with ubuntu 20.04, compile from source code
1290c0d
to
494d7e7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.