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)