Skip to content

Enable ccache on riscv64#21632

Merged
ggerganov merged 1 commit intoggml-org:masterfrom
riseproject-dev:riscv64-ccache
Apr 16, 2026
Merged

Enable ccache on riscv64#21632
ggerganov merged 1 commit intoggml-org:masterfrom
riseproject-dev:riscv64-ccache

Conversation

@luhenry
Copy link
Copy Markdown
Contributor

@luhenry luhenry commented Apr 8, 2026

Overview

This is made possible with ggml-org/ccache-action#1

Additional information

Requirements

@luhenry
Copy link
Copy Markdown
Contributor Author

luhenry commented Apr 8, 2026

cc @CISC @taronaeo

Copy link
Copy Markdown
Member

@CISC CISC left a comment

Choose a reason for hiding this comment

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

Unsure how we'll proceed, I think maybe wait for upstream to not pull in additional binaries...

Comment thread .github/workflows/build-riscv.yml Outdated
@luhenry
Copy link
Copy Markdown
Contributor Author

luhenry commented Apr 8, 2026

It’s working, without the binary from riseproject-dev/ccache. https://github.com/riseproject-dev/llama.cpp/actions/runs/24143385497/job/70475993840

@luhenry luhenry marked this pull request as ready for review April 9, 2026 21:24
@luhenry luhenry requested a review from a team as a code owner April 9, 2026 21:24
@github-actions github-actions Bot added the devops improvements to build systems and github actions label Apr 9, 2026
Comment thread .github/workflows/build-riscv.yml
@CISC CISC added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Apr 11, 2026
@ggerganov
Copy link
Copy Markdown
Member

@CISC
Copy link
Copy Markdown
Member

CISC commented Apr 14, 2026

@ggerganov I think this PR is fine to merge as-is, it targets a commit based off 1.2.21 instead of current unreleased main branch.

@luhenry
Copy link
Copy Markdown
Contributor Author

luhenry commented Apr 14, 2026

Let me bump to hendrikmuhs/ccache-action@480829a for riscv64, unless you want to add a tag?

@CISC
Copy link
Copy Markdown
Member

CISC commented Apr 14, 2026

Let me bump to hendrikmuhs/ccache-action@480829a for riscv64, unless you want to add a tag?

I'd rather not target intermediate changes, best keep it as close as possible to all the other CIs.

@ggerganov ggerganov merged commit 8612ed1 into ggml-org:master Apr 16, 2026
40 of 41 checks passed
@luhenry luhenry deleted the riscv64-ccache branch April 16, 2026 10:18
@luhenry
Copy link
Copy Markdown
Contributor Author

luhenry commented Apr 16, 2026

I’ll work on unifying Ubuntu-cpu-riscv64 into the matrix of other Ubuntu-cpu jobs next, making sure there are all the same dependencies as other platforms out of the box

cnsiva pushed a commit to saas-home/llama.cpp that referenced this pull request Apr 17, 2026
mengqin pushed a commit to mengqin/llama.cpp that referenced this pull request Apr 20, 2026
ArberSephirotheca pushed a commit to ArberSephirotheca/llama.cpp that referenced this pull request Apr 21, 2026
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Apr 23, 2026
rsenthilkumar6 pushed a commit to rsenthilkumar6/llama.cpp that referenced this pull request May 1, 2026
jimbothigpen pushed a commit to jimbothigpen/frankenturbo2 that referenced this pull request May 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants