Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Autogenerate, not autoupgrade in title
| llvm/test/CodeGen/AMDGPU/global_atomic_optimizer_fp_rtn.ll | ||
|---|---|---|
| 8 ↗ | (On Diff #549862) | this isn't a uniform pointer |
| llvm/test/CodeGen/AMDGPU/global_atomic_optimizer_fp_rtn.ll | ||
|---|---|---|
| 123 ↗ | (On Diff #550705) | If you're testing a divergent value, might as well have the pointer be divergent too |
| 136 ↗ | (On Diff #550705) | should be a separate strictfp test, the strictfp is hidden |
| 149 ↗ | (On Diff #550705) | Typo defalut. Also call it system |
| llvm/test/CodeGen/AMDGPU/global_atomics_optimizer_fp_no_rtn.ll | ||
| 149 ↗ | (On Diff #550705) | Ditto for everything in the return version |
| 165 ↗ | (On Diff #550705) | Missing space |
| llvm/test/CodeGen/AMDGPU/global_atomic_optimizer_fp_rtn.ll | ||
|---|---|---|
| 136 ↗ | (On Diff #550705) | These atomicrmw operations are tested with following attributes and there is separate attribute for strictfp which is #2 attributes #0 = { "denormal-fp-math-f32"="preserve-sign,preserve-sign" "amdgpu-unsafe-fp-atomics"="true" }
attributes #1 = { strictfp "denormal-fp-math-f32"="preserve-sign,preserve-sign" "amdgpu-unsafe-fp-atomics"="true" }
attributes #2 = { strictfp} |
| llvm/test/CodeGen/AMDGPU/global_atomic_optimizer_fp_rtn.ll | ||
|---|---|---|
| 136 ↗ | (On Diff #550705) | Which is the function I pointed to, marked with attribute group 2. The test name does not indicate this is a strictfp test and it looks like it's just testing the other combinations of uniformity and scope |
| llvm/test/CodeGen/AMDGPU/global_atomic_optimizer_fp_rtn.ll | ||
|---|---|---|
| 136 ↗ | (On Diff #550705) | you want me to mangle the attributes into function name for each of them? And Yes, these tests are the combination of fadd/fsub/fmin/fmax with different scopes and also combination of uniform or divergent input values to atomicrmw (address and value). |
you probably want to add some syncscopes to ensure they are preserved (maybe other metadata too)