(docs): update outdated macos build from source docs #269
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Explanation
This PR updates the outdated macOS build instructions page to reflect the current LLVM installation path used via Homebrew (llvm@16) and removes a deprecated section Known issues (since the issue has been resolved and there are no currently known issues).
Related issue
None
Motivation
While working on a separate PR, I noticed that the macOS build instructions were outdated and caused build failures. Specifically, the current instructions rely on
brew install llvm, which now installs LLVM 20 by default. LLVM 20 has a different header structure especially forlldwhich breaks the build with errors like:To fix this, users would need to install
lldseparately and manually update include paths, which complicates the setup. However, usingllvm@16avoids these issues because its layout matches the expectations of the WasmEdge build system (e.g., it placeslld/Common/Driver.hunder$LLVM_DIR/include).This PR updates the instructions to use
llvm@16, ensuring a smoother and consistent build experience on macOS.What type of PR is this
/kind documentation
Proposed Changes
llvm@16via Homebrew (brew install cmake ninja llvm@16) to avoid compatibility issues introduced by LLVM 20.LLVM_DIRexport path to match the location ofllvm@16CMake files:export LLVM_DIR="$(brew --prefix)/opt/llvm@16/lib/cmake".wasmedgeWasiSocketTests) has been resolved.