Update assembler tests for FLAT instructions because most opcodes have 2 tests or fewer; remove redundant gfx11_asm_flat_mnemonic.s.
Disassembler tests for FLAT instructions have already been updated.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM, thanks. Are we consistently using "flat" in the filename even though it also tests global and scratch instructions?
zero to none
:)
Comment Actions
I believe we are, though GFX10 have separate tests for these. I was not sure if splitting FLAT tests into three separate encodings would be useful.
Comment Actions
Remove now redundant gfx11_asm_flat_mnemonic.s. I somehow missed this change in the first diff.