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
@slaren
Copy link
Member

slaren commented Nov 17, 2025

Please check the CI failures.

@sudhiarm sudhiarm force-pushed the fix/17187-zero-size-array branch from dee8096 to 48800c5 Compare November 17, 2025 11:33
@sudhiarm
Copy link
Contributor Author

sudhiarm commented Nov 17, 2025

I’ve reverted {0} back to {}. The {} performs full value-initialization.
Could someone please re-run the CI to confirm it’s clean now?
@slaren

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

3 participants