aco/ra: don't use ds_write_b8_d16_hi/ds_write_b16_d16_hi on GFX8

GFX8 doesn't support these opcodes.

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Timur Kristóf <timur.kristof@gmail.com>
Fixes: c75138ed64 ("aco/ra: refactor subdword definition info")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12721>
This commit is contained in:
Rhys Perry 2021-09-03 17:15:03 +01:00 committed by Marge Bot
parent 265515fa62
commit c1e668d5d1
3 changed files with 1 additions and 7 deletions

View file

@ -7,6 +7,3 @@ dEQP-VK.wsi.*
# Exclude this test which timeout most of the time.
dEQP-VK.memory.pipeline_barrier.transfer_src_transfer_dst.1048576
# ACO hang
dEQP-VK.memory_model.shared.*

View file

@ -7,6 +7,3 @@ dEQP-VK.wsi.*
# Exclude this test which timeout most of the time.
dEQP-VK.memory.pipeline_barrier.transfer_src_transfer_dst.1048576
# ACO hang
dEQP-VK.memory_model.shared.*

View file

@ -485,7 +485,7 @@ get_subdword_operand_stride(chip_class chip, const aco_ptr<Instruction>& instr,
switch (instr->opcode) {
case aco_opcode::v_cvt_f32_ubyte0: return 1;
case aco_opcode::ds_write_b8:
case aco_opcode::ds_write_b16: return chip >= GFX8 ? 2 : 4;
case aco_opcode::ds_write_b16: return chip >= GFX9 ? 2 : 4;
case aco_opcode::buffer_store_byte:
case aco_opcode::buffer_store_short:
case aco_opcode::flat_store_byte: