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.

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

Add wamr to esp-idf components registry. #3287

Conversation

donghengqaz
Copy link
Contributor

This PR is for the main branch, but only the released branch will be pushed into Espressif component registry.

@donghengqaz
Copy link
Contributor Author

Hi @wenyongh, when I run the example in the main branch, I found the interpreter mode works well, but AOT mode shows error as follows:

E (344) wamr: Error in wasm_runtime_load: AOT module load failed: unexpect end

Do you know something about this?

@wenyongh
Copy link
Contributor

wenyongh commented Apr 7, 2024

Hi @wenyongh, when I run the example in the main branch, I found the interpreter mode works well, but AOT mode shows error as follows:

E (344) wamr: Error in wasm_runtime_load: AOT module load failed: unexpect end

Do you know something about this?

Did you build wamrc from the main branch and use it to generate the AOT file again? The AOT file format of main is different from the format of 1.3.x now.

@donghengqaz
Copy link
Contributor Author

Hi @wenyongh, when I run the example in the main branch, I found the interpreter mode works well, but AOT mode shows error as follows:

E (344) wamr: Error in wasm_runtime_load: AOT module load failed: unexpect end

Do you know something about this?

Did you build wamrc from the main branch and use it to generate the AOT file again? The AOT file format of main is different from the format of 1.3.x now.

Thanks, I will try this.

@wenyongh
Copy link
Contributor

wenyongh commented Apr 8, 2024

@donghengqaz there is a CI coding guideline check failure, could you help fix it? It seems this line is too long:

void *buf_origin = heap_caps_malloc(size + 4 + sizeof(uintptr_t), mem_caps);

It was fixed in another PR #3288

@donghengqaz
Copy link
Contributor Author

@donghengqaz there is a CI coding guideline check failure, could you help fix it? It seems this line is too long:

void *buf_origin = heap_caps_malloc(size + 4 + sizeof(uintptr_t), mem_caps);

It was fixed in another PR #3288

OK, and I will fix the AOT file data format error.

@donghengqaz donghengqaz force-pushed the tools/add_to_esp_idf_components_registry_main branch from 9b596a6 to 3a585ee Compare April 8, 2024 02:35
@donghengqaz donghengqaz force-pushed the tools/add_to_esp_idf_components_registry_main branch from 3a585ee to 5c597e5 Compare April 8, 2024 03:21
@wenyongh wenyongh merged commit dacb3c4 into bytecodealliance:main Apr 8, 2024
386 checks passed
@donghengqaz donghengqaz deleted the tools/add_to_esp_idf_components_registry_main branch April 8, 2024 06:00
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 1, 2024
This PR is for the main branch, but only the released branch will be pushed
into Espressif component registry.

See also similar fixes in branch release/1.3.x:
bytecodealliance#3264
bytecodealliance#3288
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 2, 2024
This PR is for the main branch, but only the released branch will be pushed
into Espressif component registry.

See also similar fixes in branch release/1.3.x:
bytecodealliance#3264
bytecodealliance#3288

Signed-off-by: victoryang00 <victoryang00@ucsc.edu>
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
This PR is for the main branch, but only the released branch will be pushed
into Espressif component registry.

See also similar fixes in branch release/1.3.x:
bytecodealliance#3264
bytecodealliance#3288
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
This PR is for the main branch, but only the released branch will be pushed
into Espressif component registry.

See also similar fixes in branch release/1.3.x:
bytecodealliance#3264
bytecodealliance#3288

Signed-off-by: victoryang00 <victoryang00@ucsc.edu>
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.

None yet

2 participants