Skip to content

Conversation

@sudhiarm
Copy link
Contributor

Summary
Fixes #17187.

Changes

  • Add a {} sentinel element at the end of arrays to ensure they are never zero-sized.
  • Verified build on macOS (Clang 17); portable across compilers and platforms.

@github-actions github-actions bot added the ggml changes relating to the ggml tensor library for machine learning label Nov 13, 2025
@ggerganov ggerganov requested a review from chaxu01 November 13, 2025 18:57
},
#endif
#endif
{ /* Sentinel */ }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Using {0} as an explicit initializer.

/* .op_type = */ GGML_TYPE_F32,
},
#endif
{ /* Sentinel */ }
Copy link
Collaborator

Choose a reason for hiding this comment

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

same as above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done. thanks

@sudhiarm sudhiarm force-pushed the fix/17187-zero-size-array branch from 373da48 to dee8096 Compare November 14, 2025 09:44
@sudhiarm sudhiarm requested a review from chaxu01 November 14, 2025 09:45
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.

Compile bug: Cannot compile ARM KleidiAI with GGML_CPU_ALL_VARIANTS=ON

2 participants