Skip to content

Conversation

ChinYikMing
Copy link
Collaborator

No description provided.

wget https://apt.llvm.org/llvm.sh
chmod +x ./llvm.sh
sudo ./llvm.sh 18
sudo apt-get install -q -y clang-18 clang-tools-18
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there any dependency that requires us to install clang-18 and clang-tools-18 after running llvm.sh?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Is there any dependency that requires us to install clang-18 and clang-tools-18 after running llvm.sh?

Maybe sudo ./llvm.sh 18 is enough since it includes both clang-18 and clang-tools-18.

Copy link
Collaborator Author

@ChinYikMing ChinYikMing Nov 2, 2024

Choose a reason for hiding this comment

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

The llvm.sh script seems like only adding LLVM sources list to the system. It does have the option to install all packages (e.g., sudo ./llvm.sh all). But, we only need clang-18 and clang-tools-18 during static analysis. Additionally, the system might not include LLVM 18 in its sources list, so be sure to add the LLVM 18 source list before installing them.

Please correct me if anything wrong. @jserv

@jserv jserv merged commit 27b8257 into sysprog21:master Nov 2, 2024
8 checks passed
@jserv
Copy link
Contributor

jserv commented Nov 2, 2024

Thank @ChinYikMing for contributing!

@ChinYikMing ChinYikMing deleted the ci-bump-clang branch November 2, 2024 16:42
@jserv jserv added this to the release-2024.2 milestone Nov 2, 2024
vestata pushed a commit to vestata/rv32emu that referenced this pull request Jan 24, 2025
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.

3 participants