Skip to content

Anatoliylitv/bloat 16 use hip bfloat 16 as union#3113

Draft
anatoliylitv wants to merge 2 commits intorelease/2.9from
anatoliylitv/bloat_16_use_hip__bfloat_16_as_union
Draft

Anatoliylitv/bloat 16 use hip bfloat 16 as union#3113
anatoliylitv wants to merge 2 commits intorelease/2.9from
anatoliylitv/bloat_16_use_hip__bfloat_16_as_union

Conversation

@anatoliylitv
Copy link
Copy Markdown

Motivation

Technical Details

Test Plan

Test Result

Submission Checklist

- Introduced a union in the BFloat16 struct to accommodate __hip_bfloat16 alongside uint16_t for better compatibility with HIP.
- Updated constructors and conversion operators to handle both __hip_bfloat16 and __nv_bfloat16 types based on the compilation environment.
- Adjusted conditional compilation flags to streamline the code for ROCm and CUDA environments.

This change enhances the flexibility of the BFloat16 type across different GPU architectures.
- Modified the conditional compilation directive for CUDA to remove ROCm-specific checks, ensuring compatibility with CUDA environments.
@rocm-repo-management-api
Copy link
Copy Markdown

rocm-repo-management-api bot commented Mar 31, 2026

Jenkins build for bc14f20cdbe218145ccc87b62423e65e93c337ba commit finished as FAILURE
Links: Pipeline Overview / Build artifacts / Test Results

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.

1 participant