Add mimgopc object to represent the opcode allowing different
opcodes for different hardware variants.
This enables image_atomic_fcmpswap, image_atomic_fmin, and
image_atomic_fmax on GFX10
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Looks good to me, but there are also some TODOs in test/MC/Disassembler/AMDGPU/gfx10_mimg.txt which you should be able TO DO now.
| llvm/lib/Target/AMDGPU/MIMGInstructions.td | ||
|---|---|---|
| 835 | Needs asm/dis tests? | |
| llvm/lib/Target/AMDGPU/MIMGInstructions.td | ||
|---|---|---|
| 835 | I have added asm tests. | |
Comment Actions
- Implement test/MC/Disassembler/AMDGPU/gfx10_mimg.txt TODOs
- Add GFX7 asm tests for image_atomic_rsub
| llvm/lib/Target/AMDGPU/MIMGInstructions.td | ||
|---|---|---|
| 835 | Sure. You would have to add a gfx7 disassembler first :) | |
Needs asm/dis tests?