Skip to content

Conversation

wffurr
Copy link

@wffurr wffurr commented Oct 1, 2025

This required minor changes to LLVMContext construction and PGOOptions.

"repo": "https://github.com/espressif/llvm-project.git",
"repo_ssh": "[email protected]:espressif/llvm-project.git",
"branch": "xtensa_release_18.1.2",
"branch": "xtensa_release_21.1.2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no such a branch availabile

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - I should have checked that. Looks like the new API for LLVMContext construction will have to get wrapped in an #ifdef LLVM version check, since xtensa's latest release is only LLVM 19.

@wffurr wffurr force-pushed the update-llvm branch 2 times, most recently from c11c325 to 0a7e444 Compare October 2, 2025 18:03
Wraps the LLVMContext construction in LLVM 21 version check; LLVM 21 makes a
breaking change in LLVMContext construction, but WAMR still needs to
support older LLVM versions, e.g. for xtensa/esp32 support which is only
available in at most LLVM 19.
@wffurr
Copy link
Author

wffurr commented Oct 3, 2025

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!

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.

2 participants