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

LLVM ERROR: failed to perform tail call elimination with MIPS AOT #2412

Closed
eloparco opened this issue Jul 31, 2023 · 3 comments · Fixed by #2457
Closed

LLVM ERROR: failed to perform tail call elimination with MIPS AOT #2412

eloparco opened this issue Jul 31, 2023 · 3 comments · Fixed by #2457

Comments

@eloparco
Copy link
Contributor

Hello, I'm getting this error when trying to compile for MIPS:

$ wasm-micro-runtime/wamr-compiler/build/wamrc --enable-multi-thread --disable-simd --target=mips --size-level=1 -o pigz.aot pigz.wasm
Create AoT compiler with:
  target:        mips
  target cpu:    
  target triple: mips-pc-linux-gnu
  cpu features:  
  opt level:     3
  size level:    1
  output format: AoT file
LLVM ERROR: failed to perform tail call elimination on a call site marked musttail
78805 Abort trap: 6           wasm-micro-runtime/wamr-compiler/build/wamrc --enable-multi-thread --disable-simd --target=mips --size-level=1 -o "pigz.aot" "pigz.wasm"

The wasm file is pigz.wasm.zip and it's coming from https://github.com/bytecodealliance/wasm-parallel-gzip.
It works fine with other targets (I tried aarch64, armv7, x86_64).

@eloparco
Copy link
Contributor Author

eloparco commented Aug 1, 2023

I just checked out version 1.2.2 and there the compilation is fine, so the issue was introduced after that.

@loganek
Copy link
Collaborator

loganek commented Aug 1, 2023

I just did a quick git bisect check; looks like the PR #2244 caused the regression.

yamt added a commit to yamt/wasm-micro-runtime that referenced this issue Aug 14, 2023
While i'm not familiar with mips abi, it's reported that
it (at least sometimes) doesn't work.
This fixes bytecodealliance#2412
@yamt
Copy link
Collaborator

yamt commented Aug 14, 2023

i tested #2457 with the given wasm file lightly.
(lightly == just aot compile. i have no mips env.)

wenyongh pushed a commit that referenced this issue Aug 15, 2023
Zzzabiyaka pushed a commit to Zzzabiyaka/wasm-micro-runtime that referenced this issue Aug 15, 2023
Zzzabiyaka pushed a commit to Zzzabiyaka/wasm-micro-runtime that referenced this issue Aug 15, 2023
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this issue May 27, 2024
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 a pull request may close this issue.

3 participants