Skip to content

[AMDGPU][True16] turn on true16 for all gfx11 devices #143518

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

Merged
merged 1 commit into from
Jul 21, 2025

Conversation

broxigarchen
Copy link
Contributor

@broxigarchen broxigarchen commented Jun 10, 2025

A follow up patch from #140736. Set default true16 mode from gfx110x to all gfx11 devices.

Tests has been address in preivous patches.

@broxigarchen broxigarchen changed the title turn on true16 for all gfx11 devices [AMDGPU][True16] turn on true16 for all gfx11 devices Jun 10, 2025
@broxigarchen broxigarchen marked this pull request as ready for review June 10, 2025 14:12
@llvmbot
Copy link
Member

llvmbot commented Jun 10, 2025

@llvm/pr-subscribers-backend-amdgpu

Author: Brox Chen (broxigarchen)

Changes

set default true16 mode for all gfx11 devices


Full diff: https://github.com/llvm/llvm-project/pull/143518.diff

1 Files Affected:

  • (modified) llvm/lib/Target/AMDGPU/AMDGPU.td (+3-3)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPU.td b/llvm/lib/Target/AMDGPU/AMDGPU.td
index a9b39eebbcdcf..d1f0a49e66aa2 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPU.td
+++ b/llvm/lib/Target/AMDGPU/AMDGPU.td
@@ -1780,7 +1780,8 @@ def FeatureISAVersion11_Common : FeatureSet<
    FeatureImageInsts,
    FeaturePackedTID,
    FeatureVcmpxPermlaneHazard,
-   FeatureMemoryAtomicFAddF32DenormalSupport]>;
+   FeatureMemoryAtomicFAddF32DenormalSupport,
+   FeatureRealTrue16Insts]>;
 
 // There are few workarounds that need to be
 // added to all targets. This pessimizes codegen
@@ -1800,8 +1801,7 @@ def FeatureISAVersion11_0_Common : FeatureSet<
     [FeatureMSAALoadDstSelBug,
      FeatureVALUTransUseHazard,
      FeatureMADIntraFwdBug,
-     FeaturePrivEnabledTrap2NopBug,
-     FeatureRealTrue16Insts])>;
+     FeaturePrivEnabledTrap2NopBug])>;
 
 def FeatureISAVersion11_0_0 : FeatureSet<
   !listconcat(FeatureISAVersion11_0_Common.Features,

@broxigarchen broxigarchen merged commit 0c804da into llvm:main Jul 21, 2025
11 checks passed
searlmc1 pushed a commit to ROCm/llvm-project that referenced this pull request Jul 22, 2025
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Jul 28, 2025
A follow up patch from llvm#140736.
Set default true16 mode from gfx110x to all gfx11 devices.

Tests has been address in preivous patches.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants