Skip to content

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

Merged
ggerganov merged 2 commits into
ggml-org:masterfrom
taronaeo:ci/s390x-non-vxe
Sep 14, 2026
Merged

ggerganov merged 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 d4645257b6b7e65c47b1b46baec3eb46a3f40968.

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
@ggerganov
ggerganov merged commit be2c6d7 into ggml-org:master Sep 14, 2026
30 of 31 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Sep 14, 2026
1 task done
quimmedes pushed a commit to quimmedes/cafe-llama.cpp that referenced this pull request Sep 16, 2026
* tests: add non-vxe build to tests

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 d4645257b6b7e65c47b1b46baec3eb46a3f40968.

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

* ggml-cpu: revert back to upstream/master

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

---------

Signed-off-by: Aaron Teo <aaron.teo1@ibm.com>
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.

3 participants