Uh oh!
There was an error while loading. Please reload this page.
fix: resolve const-correctness error in ggml-bitnet-mad.cpp - #409
Open
Eon (eon-ic) wants to merge 1 commit into
Open
fix: resolve const-correctness error in ggml-bitnet-mad.cpp#409Eon (eon-ic) wants to merge 1 commit into
Eon (eon-ic) wants to merge 1 commit into
Conversation
Edward Larson (EdwardLarson)
commented
Feb 18, 2026
+1 I had the same issue and this fixed it. tsong-ms PTAL 🙏 |
| } | ||
| int8_t * y_col = y + col * by; | ||
| const int8_t * y_col = y + col * by; |
There was a problem hiding this comment.
The following worked for me:
Suggested change
| constint8_t * y_col = y + col * by; | |
| int8_t * y_col = (int8_t *)(y + col * by); |
But yours also should work off course.
Jacob Leksan (jmleksan)
commented
Mar 3, 2026
I have the same issue! |
Andrew Barnes (Bortlesboat)
commented
Mar 13, 2026
Validated this locally as part of a Windows LLVM/x64 build pass. Environment:
What I observed:
Changing it to:
resolved that compile error and allowed the build to continue. So from another independent reproduction: this PR fixes a real current blocker, not just an older environment-specific failure. |
Adrian (adrianhoehne)
commented
Apr 6, 2026
Thank you. I also run into that problem today. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the following build error:
ggml-bitnet-mad.cpp:811:17: error: cannot initialize a variable of type 'int8_t *' with an rvalue of type 'const int8_t *'
Changed
int8_t *y_coltoconst int8_t *y_colto respect const correctness, asyis a const input pointer and its data should not be modified.