From: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Date: Wed, 19 Aug 2020 11:42:57 +0000 (+0200)
Subject: radv: ignore BB labels when splitting the disassembly string
X-Git-Url: https://git.libre-soc.org/?p=mesa.git;a=commitdiff_plain;h=f811ddf5d1fac8a857aadc51278546d5b46d5a2b

radv: ignore BB labels when splitting the disassembly string

Ignore everything that is not an instruction. This should fix
the annotated UMR disassembly.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6387>
---

diff --git a/src/amd/vulkan/radv_debug.c b/src/amd/vulkan/radv_debug.c
index e86b0079df7..47d70b96499 100644
--- a/src/amd/vulkan/radv_debug.c
+++ b/src/amd/vulkan/radv_debug.c
@@ -273,6 +273,12 @@ static void si_add_split_disasm(const char *disasm,
 		struct radv_shader_inst *inst = &instructions[*num];
 		unsigned len = next - disasm;
 
+		if (!memchr(disasm, ';', len)) {
+			/* Ignore everything that is not an instruction. */
+			disasm = next + 1;
+			continue;
+		}
+
 		assert(len < ARRAY_SIZE(inst->text));
 		memcpy(inst->text, disasm, len);
 		inst->text[len] = 0;