Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Looks good.
| llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp | ||
|---|---|---|
| 3298 | How do we handle the addressing mode where there is no saddr, just vaddr+offset? | |
| llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp | ||
|---|---|---|
| 3298 | Yes. The case this doesn't handle (which I haven't done in the DAG either) is: (SGPR64 + imm), which does require initializing the VGPR offset to 0, but this is fewer instructions than the two required to copy an SGPR64 to VGPR64 | |
Comment Actions
Rebased and moved IR test changes to shift optimization patch a9ee0589a8bc1584af4209fe6439c68aa875065f
How do we handle the addressing mode where there is no saddr, just vaddr+offset?