Skip to content

Conversation

@jplehr
Copy link
Contributor

@jplehr jplehr commented Aug 14, 2025

It slipped my attention that this builder was (still?) doing incremental builds. This is highly not recommended by the community.

This patch nukes an existing build dir completely to start from a clean state and rely on ccache to bring build-time improvements.

It slipped my attention that this builder was (still?) doing incremental
builds. This is highly not recommended by the community.

This patch nukes an existing build dir completely to start from a clean
state and rely on ccache to bring build-time improvements.
Copy link
Contributor

@rovka rovka left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!

@jplehr jplehr merged commit 8aa546f into llvm:main Aug 14, 2025
3 checks passed
@jplehr jplehr deleted the fix/prefer-ccache-over-incremental-build branch August 14, 2025 14:46
vvereschaka pushed a commit to vvereschaka/llvm-zorg that referenced this pull request Sep 25, 2025
It slipped my attention that this builder was (still?) doing incremental
builds. This is highly not recommended by the community.

This patch nukes an existing build dir completely to start from a clean
state and rely on ccache to bring build-time improvements.
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