bytecodealliance
Update LLVM to 21.1.2.
Is this good to merge? The test failures look like timeouts; one error is "fatal error: error writing to /tmp/cckAF38s.s: No space left on device" which doesn't seem relevant. Thanks!
Is this good to merge? The test failures look like timeouts; one error is "fatal error: error writing to /tmp/cckAF38s.s: No space left on device" which doesn't seem relevant. Thanks!
isn't it possible llvm 21 build actually somehow requires more space and caused ENOSPC on the ci? i guess someone needs to investigate a bit further. (sorry i have no time right now)
Is this good to merge? The test failures look like timeouts; one error is "fatal error: error writing to /tmp/cckAF38s.s: No space left on device" which doesn't seem relevant. Thanks!
isn't it possible llvm 21 build actually somehow requires more space and caused ENOSPC on the ci? i guess someone needs to investigate a bit further. (sorry i have no time right now)
I found a script to free up disk space on the GitHub Ubuntu runner. Is that OK to use? Seems to fix the out of space error. If you'd prefer I didn't use the action from the marketplace, I can hoist that bit out into a script in the WAMR repo and use that instead.
@yamt @TianlongLiang @loganek What are your thoughts on upgrading LLVM to version 21?
@yamt @TianlongLiang @loganek What are your thoughts on upgrading LLVM to version 21?
i don't have strong opinions either ways. it's a bit sad to use different versions for different targets. but i guess it doesn't matter much. otoh, i don't see any strong reasons to update to 21 either.
For ARC, I don't think we should upgrade LLVM to 21.x. For the default target, we do need to update static pgo-related data structures like in this pr. Personally, I think we need to update the static PGO first before actually upgrading LLVM.
We're using WAMR with LLVM head and have patches for LLVM's changes in 21 and 22 that break WAMR. This is probably not the way the software is intended to be used. I was hoping to update to something closer to head.
I could send the patch for just #if LLVM_VERSION_MAJOR >= 21 in aot_llvm.c instead of updating the LLVM targets in the build script.