From: Rhys Perry Date: Wed, 5 Aug 2020 13:59:01 +0000 (+0100) Subject: aco: shorten disassembly for repeated instructions X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=192b9f43039eee21288145e940502de5eca1a69b;p=mesa.git aco: shorten disassembly for repeated instructions Future tests will do this. Signed-off-by: Rhys Perry Reviewed-by: Daniel Schürmann Part-of: --- diff --git a/src/amd/compiler/aco_print_asm.cpp b/src/amd/compiler/aco_print_asm.cpp index d4b0e0edbcf..ce12aacf9e4 100644 --- a/src/amd/compiler/aco_print_asm.cpp +++ b/src/amd/compiler/aco_print_asm.cpp @@ -146,38 +146,52 @@ void print_asm(Program *program, std::vector& binary, binary[pos+1] = binary[pos+1] & 0xF803FFFF; } + const int align_width = 60; + out << std::left << std::setw(align_width) << std::setfill(' '); + size_t l = LLVMDisasmInstruction(disasm, (uint8_t *) &binary[pos], (exec_size - pos) * sizeof(uint32_t), pos * 4, outline, sizeof(outline)); size_t new_pos; - const int align_width = 60; if (!l && ((program->chip_class >= GFX9 && (binary[pos] & 0xffff8000) == 0xd1348000) || /* v_add_u32_e64 + clamp */ (program->chip_class >= GFX10 && (binary[pos] & 0xffff8000) == 0xd7038000) || /* v_add_u16_e64 + clamp */ (program->chip_class <= GFX9 && (binary[pos] & 0xffff8000) == 0xd1268000)) /* v_add_u16_e64 + clamp */) { - out << std::left << std::setw(align_width) << std::setfill(' ') << "\tinteger addition + clamp"; + out << "\tinteger addition + clamp"; bool has_literal = program->chip_class >= GFX10 && (((binary[pos+1] & 0x1ff) == 0xff) || (((binary[pos+1] >> 9) & 0x1ff) == 0xff)); new_pos = pos + 2 + has_literal; } else if (program->chip_class >= GFX10 && l == 4 && ((binary[pos] & 0xfe0001ff) == 0x020000f9)) { - out << std::left << std::setw(align_width) << std::setfill(' ') << "\tv_cndmask_b32 + sdwa"; + out << "\tv_cndmask_b32 + sdwa"; new_pos = pos + 2; } else if (!l) { - out << std::left << std::setw(align_width) << std::setfill(' ') << "(invalid instruction)"; + out << "(invalid instruction)"; new_pos = pos + 1; invalid = true; } else { - out << std::left << std::setw(align_width) << std::setfill(' ') << outline; + out << outline; assert(l % 4 == 0); new_pos = pos + l / 4; } + size_t size = new_pos - pos; out << std::right; out << " ;"; - for (; pos < new_pos; pos++) - out << " " << std::setfill('0') << std::setw(8) << std::hex << binary[pos]; + for (unsigned i = 0; i < size; i++) + out << " " << std::setfill('0') << std::setw(8) << std::hex << binary[pos + i]; out << std::endl; + + size_t original_pos = pos; + pos += size; + + unsigned repeat_count = 0; + while (pos + size <= exec_size && memcmp(&binary[pos], &binary[original_pos], size * 4) == 0) { + repeat_count++; + pos += size; + } + if (repeat_count) + out << std::left << std::setw(0) << std::dec << std::setfill(' ') << "\t(then repeated " << repeat_count << " times)" << std::endl; } out << std::setfill(' ') << std::setw(0) << std::dec; assert(next_block == program->blocks.size());