llvm: Cleanups and improvements #535
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously using `static-libgcc` with Clang would fail with an error that `libunwind.a` couldn't be found. This is because we have a static libgcc build, but not a corresponding static libunwind. Ensure that we build said static libunwind. Signed-off-by: Reilly Brogan <[email protected]>
Changes: - Shrink the 32bit build significantly by limiting the target support to X86 and AMDGPU, and also only building LLVM. Also disable several additional features not needed for the 32bit build - And ARM and WebAssembly target support to the 64bit build for users cross-compiling - Get ccache working properly, and add debugging information so that it's possible to see how effective it is. - Disable building with PGO (this is temporary and will be re-enabled post LLVM 19) - Add openmp support (libomp) - Add clang-tools-extra (clang-tidy plus many other tools) so that we can build a more generally useful LLVM distribution for users. - Create `clang-tools` subpackage and move all of the build artifacts of `clang-tools-extra` there as well as clang-analyzer and other tools that were in `clang` that were not strictly necessary for compilation - Ensure that subpackage rundeps effectively pull in the packages they need. For instance `clang-devel` now depends on `llvm-devel` - Update various package descriptions - Build lld, clang, and llvm-ar so that they use the mimalloc memory allocator (improves performance by ~7%) - Patch LLD so that it always behaves as though `--build-id=sha1` was passed (this matches binutils/mold) - Patch LLD so that it always behaves as though `--compress-debug-symbols=zstd` was passed (this matches binutils/mold) Signed-off-by: Reilly Brogan <[email protected]>
ReillyBrogan
force-pushed
the
llvm-cleanup
branch
from
557782d to
8644106
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changes:
clang-toolssubpackage and move all of the build artifacts ofclang-tools-extrathere as well as clang-analyzer and other tools that were inclangthat were not strictly necessary for compilationclang-develnow depends onllvm-devel--build-id=sha1was passed (this matches binutils/mold)--compress-debug-symbols=zstdwas passed (this matches binutils/mold)Depends on serpent-os/tools#402
Depends on #447
Depends on serpent-os/tools#381 (not technically required but I only tested this with that PR applied)