These intrinsics are missing mangling for both the pointer and data
type.
Details
Diff Detail
Event Timeline
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | ||
---|---|---|
3225 | "Helper" can be marked as "const"? | |
3232 | 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 | ||
3 | Why just mesa3d? No implication on hsa? |
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | ||
---|---|---|
3225 | It wouldn't be if it were used as normal | |
3232 | 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 | ||
3 | 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"?