mirror of
https://github.com/deepseek-ai/DeepGEMM.git
synced 2026-11-05 00:47:26 +00:00
[GH-ISSUE #32] Use f16 accumulator instead of f32 #21
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
deepseek-ai/DeepGEMM#21
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @i-zhen on GitHub (Mar 3, 2025).
Original GitHub issue: https://github.com/deepseek-ai/DeepGEMM/issues/32
Hi, thanks a lot for your great contributions.
I just wonder that why a full-precision F32 accumulator is used instead of F16. Such as following assembly code:
Can we also achieve an acceptable precision with f16 accumulator?
@LyricZhao commented on GitHub (Mar 3, 2025):
Even FP32 accumulator is not enough (that's why we use two-level accumulation: tensor core (low-precision FP32 accum, hardward "bugs") + CUDA core (full FP32 accum)).
"Not enough" means bad for training.
However, for inference, I guess it is related to different model settings: one-level tensor core FP32 accum (works for DeepSeek-V2 series), or one-level tensor core FP16 (not tested on any model, you can try by yourself).
@i-zhen commented on GitHub (Mar 4, 2025):
Thank you for your kind reply~