From e392603d8126f8d7de0c31ae366dc668d90868ab Mon Sep 17 00:00:00 2001 From: Matthew Poremba Date: Mon, 26 Dec 2022 09:08:25 -0800 Subject: [PATCH] arch-vega: Add missing operand size for ds_write2st64_b64 This instruction takes three operands (address, and two datas) but there were only operand sizes for two operands tripping assert in default case. Change-Id: I3f505b6432aee5f3f265acac46b83c0c7daff3e7 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67071 Maintainer: Matt Sinclair Tested-by: kokoro Reviewed-by: Matt Sinclair --- src/arch/amdgpu/vega/insts/instructions.hh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/arch/amdgpu/vega/insts/instructions.hh b/src/arch/amdgpu/vega/insts/instructions.hh index 0671df8452..1c422481de 100644 --- a/src/arch/amdgpu/vega/insts/instructions.hh +++ b/src/arch/amdgpu/vega/insts/instructions.hh @@ -33553,7 +33553,9 @@ namespace VegaISA switch (opIdx) { case 0: //vgpr_a return 4; - case 1: //vgpr_d1 + case 1: //vgpr_d0 + return 8; + case 2: //vgpr_d1 return 8; default: fatal("op idx %i out of bounds\n", opIdx);