Skip to content

tests(s390x): add non-vxe build to tests - #28776

Open
taronaeo wants to merge 2 commits into
ggml-org:masterfrom
taronaeo:ci/s390x-non-vxe
Open

tests(s390x): add non-vxe build to tests#28776
taronaeo wants to merge 2 commits into
ggml-org:masterfrom
taronaeo:ci/s390x-non-vxe

Conversation

@taronaeo

Copy link
Copy Markdown
Member

Overview

cont #28667

Additional information

Although we write code mainly for -DGGML_VXE=ON, there are instances where vector intrinsic is not compiled in. Our current CI setup does not catch bugs for -DGGML_VXE=OFF and this PR fixes that by adding another test in the matrix to compile and catch these problems early.

Requirements

@taronaeo
taronaeo requested a review from a team as a code owner September 11, 2026 18:41
@github-actions github-actions Bot added the devops improvements to build systems and github actions label Sep 11, 2026

@CISC CISC left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This was clearly necessary. :)

@taronaeo
taronaeo requested a review from ggerganov as a code owner September 13, 2026 13:44
@github-actions github-actions Bot added the ggml changes relating to the ggml tensor library for machine learning label Sep 13, 2026
@taronaeo

taronaeo commented Sep 13, 2026

Copy link
Copy Markdown
Member Author

Okay s390x CIs are green now but #28775 is not approved yet to move this forward :(

Edit: Yay we're almost there

Signed-off-by: Aaron Teo <aaron.teo1@ibm.com>

ggml-cpu: add unused macro to fix ci

Signed-off-by: Aaron Teo <aaron.teo1@ibm.com>

Revert "ggml-cpu: temporarily add ggml-org#28775 patch until its merged"

This reverts commit d464525.

Signed-off-by: Aaron Teo <aaron.teo1@ibm.com>
Signed-off-by: Aaron Teo <aaron.teo1@ibm.com>
@taronaeo taronaeo added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Sep 13, 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 ggml changes relating to the ggml tensor library for machine learning 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.

2 participants