[llvm] [AMDGPU] Remove wavefrontsize feature from GFX10+ (PR #98400)

Stanislav Mekhanoshin via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 00:20:22 PDT 2024


================
@@ -105,6 +105,14 @@ GCNSubtarget::initializeSubtargetDependencies(const Triple &TT,
                                         : AMDGPUSubtarget::SOUTHERN_ISLANDS;
   }
 
+  if (!hasFeature(AMDGPU::FeatureWavefrontSize32) &&
+      !hasFeature(AMDGPU::FeatureWavefrontSize64)) {
+    if (getGeneration() >= AMDGPUSubtarget::GFX10)
+      ToggleFeature(AMDGPU::FeatureWavefrontSize32);
+    else
+      ToggleFeature(AMDGPU::FeatureWavefrontSize64);
----------------
rampitec wrote:

We still need to select default somewhere. So say a subtarget has FeatureSupportsWave32, then what?

https://github.com/llvm/llvm-project/pull/98400


More information about the llvm-commits mailing list