KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

pan/va: Dump unencodable instructions 

When we assert out due to certain invalid encoding, it's helpful to know what
instruction is causing the failure, since it may not be obvious from the
assembly for large shaders. Now we get nice errors when failing:

   Invalid opcode:
      br0 = VAR_TEX.f32.flow8.store.skip.lod_mode.center , texture_index:0, varying_index:0

Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17224>
This commit is contained in:
Alyssa Rosenzweig 2022-06-22 17:01:02 -04:00 committed by Marge Bot
parent fd1edbc6e5
commit cc94409d70
1 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/panfrost/bifrost/valhall/va_pack.c
View File

@ -31,6 +31,19 @@
* bits on the wire (as well as fixup branches)
*/
/*
* Unreachable for encoding failures, when hitting an invalid instruction.
* Prints the (first) failing instruction to aid debugging.
*/
NORETURN static void
invalid_instruction(const bi_instr *I, const char *cause)
{
fprintf(stderr, "\nPacking failure due to invalid %s:\n\t", cause);
bi_print_instr(I, stderr);
fprintf(stderr, "\n");
unreachable("Invalid instruction");
}
/*
* Validate that two adjacent 32-bit sources form an aligned 64-bit register
* pair. This is a compiler invariant, required on Valhall but not on Bifrost.
@ -277,7 +290,7 @@ va_pack_source_format(const bi_instr *I)
case BI_SOURCE_FORMAT_F16: return VA_SOURCE_FORMAT_SRC_F16;
}
unreachable("unhandled source format");
invalid_instruction(I, "source format");
}
static uint64_t
@ -480,8 +493,8 @@ va_pack_alu(const bi_instr *I)
assert(src_info.size == VA_SIZE_8);
assert(i == 0);
hex |= (uint64_t) va_pack_halfswizzle(src.swizzle) << 36;
} else {
assert(src.swizzle == BI_SWIZZLE_H01 && "Unexpected swizzle");
} else if (src.swizzle != BI_SWIZZLE_H01) {
invalid_instruction(I, "swizzle");
}
}
@ -613,7 +626,7 @@ va_pack_register_format(const bi_instr *I)
case BI_REGISTER_FORMAT_S16: return VA_REGISTER_FORMAT_S16;
case BI_REGISTER_FORMAT_U32: return VA_REGISTER_FORMAT_U32;
case BI_REGISTER_FORMAT_U16: return VA_REGISTER_FORMAT_U16;
default: unreachable("unhandled register format");
default: invalid_instruction(I, "register format");
}
}
@ -776,11 +789,8 @@ va_pack_instr(const bi_instr *I)
}
default:
if (!info.exact && I->op != BI_OPCODE_NOP) {
bi_print_instr(I, stderr);
fflush(stderr);
unreachable("Opcode not packable on Valhall");
}
if (!info.exact && I->op != BI_OPCODE_NOP)
invalid_instruction(I, "opcode");
hex |= va_pack_alu(I);
break;