-
Notifications
You must be signed in to change notification settings - Fork 769
[Joint Matrix] Enable different accumulator and output types in spirv. Add tests to cover bfloat16 and half floating-point sizes. #17502
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: sycl
Are you sure you want to change the base?
Conversation
f5e46ba
to
3a5de5a
Compare
3a5de5a
to
c5be2c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need to change the test to separate acccumulator from desctination. There are 4 cases:
acc bf16, dest float
acc bf16, dest bf16
acc float, dest float
acc float, dest bf16
c5be2c5
to
29c35ac
Compare
29c35ac
to
8116625
Compare
8116625
to
fc7eaf7
Compare
// UNSUPPORTED: target-nvidia, target-amd | ||
// UNSUPPORTED-INTENDED: aspect-ext_intel_matrix isn't currently supported for | ||
// other triples | ||
|
||
// REQUIRES: aspect-ext_intel_matrix | ||
// REQUIRES-INTEL-DRIVER: lin: 27868, win: 101.5181 | ||
// REQUIRES-INTEL-DRIVER: lin: 27501, win: 101.4943 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this change?
sycl/test-e2e/Matrix/joint_matrix_bfloat16_PackedB_Accumulator.cpp
Outdated
Show resolved
Hide resolved
sycl/test-e2e/Matrix/joint_matrix_bfloat16_PackedB_Accumulator.cpp
Outdated
Show resolved
Hide resolved
// other triples | ||
|
||
// REQUIRES: aspect-ext_intel_matrix | ||
// REQUIRES-INTEL-DRIVER: lin: 27501, win: 101.4943 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this version of driver required?
sycl/test-e2e/Matrix/joint_matrix_bfloat16_PackedB_Accumulator.cpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
provided review.
5279c10
to
1bdb402
Compare
1bdb402
to
febd44d
Compare
febd44d
to
460ea6e
Compare
…sting bfloat16 to float
460ea6e
to
918c453
Compare
Updated matrix_compare and matrix_multiply_ref functions to better match bfloat16 calculations on device.