Adds patterns for f64 ds_min/ds_max. Shrinks HasLDSFPAtomics
scope to enable f32.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Shrinks HasLDSFPAtomics scope to enable f32.
"... to enable f32 add"? Maybe update the description of the feature to reflect this?
llvm/test/CodeGen/AMDGPU/lds-atomic-add.ll | ||
---|---|---|
1 | This file looks like a subset of the existing test/CodeGen/AMDGPU/lds_atomic_f32.ll? |
Agreed
llvm/test/CodeGen/AMDGPU/lds-atomic-add.ll | ||
---|---|---|
1 | I split the test because the lds_atomic_fadd is only on gfx8 and gfx9, but the min/max are available on all architectures |
llvm/test/CodeGen/AMDGPU/lds-atomic-add.ll | ||
---|---|---|
1 | Oh, I hadn't even noticed that your patch deletes lds_atomic_f32.ll. OK then. The new files should probably be called lds-atomic-fadd.ll and lds-atomic-fmin-fmax.ll. |
This file looks like a subset of the existing test/CodeGen/AMDGPU/lds_atomic_f32.ll?