Skip to content
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

Enable BFloat16 and TensorFloat32 conversions for cooperative matrices #2213

Merged
merged 3 commits into from
Nov 16, 2023

Conversation

MrSidims
Copy link
Contributor

Previously added scalar/vector ConvertFToBF16INTEL, ConvertBF16ToFINTEL and RoundFToTF32INTEL conversions are now enabled for cooperative matrix type under SPV_INTEL_joint_matrix extension following the spec: https://github.com/intel/llvm/blob/sycl/sycl/doc/design/spirv-extensions/SPV_INTEL_joint_matrix.asciidoc

Note, joint matrices are not allowed as input/output for these conversions as it is being deprecated.

@MrSidims
Copy link
Contributor Author

@VyacheslavLevytskyy please take a look

@VyacheslavLevytskyy
Copy link
Contributor

@MrSidims It's interesting that we have now two more cases when std::optional<ExtensionID> getRequiredExtension() is expected to return a vector of extensionID's, in addition to the SPV_EXT_shader_atomic_float16_add case.

@MrSidims
Copy link
Contributor Author

Restarting CI

@MrSidims MrSidims closed this Nov 15, 2023
@MrSidims MrSidims reopened this Nov 15, 2023
@MrSidims MrSidims closed this Nov 16, 2023
@MrSidims MrSidims reopened this Nov 16, 2023
@MrSidims MrSidims force-pushed the convert-matrix branch 2 times, most recently from b681e1d to f327d4b Compare November 16, 2023 01:17
Previously added scalar/vector ConvertFToBF16INTEL, ConvertBF16ToFINTEL
and RoundFToTF32INTEL conversions are now enabled for cooperative matrix
type under SPV_INTEL_joint_matrix extension following the spec:
https://github.com/intel/llvm/blob/sycl/sycl/doc/design/spirv-extensions/SPV_INTEL_joint_matrix.asciidoc

Note, joint matrices are not allowed as input/output for these
conversions as it is being deprecated.

Signed-off-by: Sidorov, Dmitry <dmitry.sidorov@intel.com>
Signed-off-by: Sidorov, Dmitry <dmitry.sidorov@intel.com>
Signed-off-by: Sidorov, Dmitry <dmitry.sidorov@intel.com>
@MrSidims MrSidims merged commit 1010efc into KhronosGroup:main Nov 16, 2023
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants