ci,scripts: install LLVM before cargo tools#14060
Conversation
Reorder bootstrap and install_build_tools.sh so LLVM 19 is installed before cargo tools. This is needed because install_cargo_tools.sh will install starknet-native-compile which requires LLVM to build. Safe change: LLVM installation does not depend on Rust. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
PR SummaryMedium Risk Overview This reorders Reviewed by Cursor Bugbot for commit 23ee70d. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Artifacts upload workflows: |
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 2 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on avi-starkware).
Reorder bootstrap and install_build_tools.sh so LLVM 19 is installed
before cargo tools. This is needed because install_cargo_tools.sh will
install starknet-native-compile which requires LLVM to build.
Safe change: LLVM installation does not depend on Rust.
Re-do of #13674, which was accidentally merged into
avi/compiler-verify-fninstead of
main. The change itself (squash commit08764c4529) is identical;this PR cherry-picks that commit onto
main.Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com