These intrinsics are missing mangling for both the pointer and data
type.
Details
Diff Detail
Event Timeline
| llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | ||
|---|---|---|
| 3227 | "Helper" can be marked as "const"? | |
| 3234 | You could probably push this inside loop, run loop from 1 to 5 and skip 2 with a comment. That way it would be readable. If its possible then an uptick loop, please. | |
| llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.ds.fadd.ll | ||
| 2 | Why just mesa3d? No implication on hsa? | |
| llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | ||
|---|---|---|
| 3227 | It wouldn't be if it were used as normal | |
| 3234 | removing an operand in the middle of an instruction will force the instruction to shift all of its operands down each time, so it's much less efficient | |
| llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.ds.fadd.ll | ||
| 2 | The triple is irrelevant, but we error on the graphics shader conventions for HSA. We use the shaders for the simplest way to get sample SGPRs | |
"Helper" can be marked as "const"?