Skip to content

Conversation

@mehendarkarprajwal
Copy link
Contributor

With recent redesign of ggml-cpu we see that it was failing to compile on POWER arch on both ppcle64 and AIX. The changes in this PR is for fixing the compilation issue.

@github-actions github-actions bot added the ggml changes relating to the ggml tensor library for machine learning label Apr 8, 2025
vec_extract_fp32_from_shorth(vec_xl(0, p - GGML_F16_EPR)) : \
vec_extract_fp32_from_shortl(vec_xl(0, p))
#define GGML_ENDIAN_BYTE(i) ((unsigned char *)&(uint16_t){1})[i]
static inline unsigned char ggml_endian_byte(int i){
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
static inline unsigned char ggml_endian_byte(int i){
static inline unsigned char ggml_endian_byte(int i) {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated files according to reviewed suggestions. Thank you!!

@slaren slaren merged commit 11d07e1 into ggml-org:master Apr 9, 2025
51 checks passed
colout pushed a commit to colout/llama.cpp that referenced this pull request Apr 21, 2025
* Including limits file on AIX

* Fixes ggml-org#12823
timwu pushed a commit to timwu/llama.cpp that referenced this pull request May 5, 2025
* Including limits file on AIX

* Fixes ggml-org#12823
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ggml changes relating to the ggml tensor library for machine learning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants