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?