From: Tuan Ta <taquangtuan1992@gmail.com>
Date: Sat, 18 Jun 2016 17:02:13 +0000 (-0400)
Subject: gpu-compute: Fixed a bug in decoding Atomic ST
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=bb9033c26b6a84f4593c020a8078101e17a7b124;p=gem5.git

gpu-compute: Fixed a bug in decoding Atomic ST

There is a mismatch between DataType and SrcDataType in constructing
Atomic ST instruction. The mismatch causes atomic_store and
atomic_store_explicit function to store incorrect value in memory.
---

diff --git a/src/arch/hsail/insts/mem.hh b/src/arch/hsail/insts/mem.hh
index 1db98d212..acc8434be 100644
--- a/src/arch/hsail/insts/mem.hh
+++ b/src/arch/hsail/insts/mem.hh
@@ -1556,11 +1556,11 @@ namespace HsailISA
               case Brig::BRIG_TYPE_B8:
                 return decodeSt<S8,S8>(ib, obj);
               case Brig::BRIG_TYPE_B16:
-                return decodeSt<S8,S16>(ib, obj);
+                return decodeSt<S16,S16>(ib, obj);
               case Brig::BRIG_TYPE_B32:
-                return decodeSt<S8,S32>(ib, obj);
+                return decodeSt<S32,S32>(ib, obj);
               case Brig::BRIG_TYPE_B64:
-                return decodeSt<S8,S64>(ib, obj);
+                return decodeSt<S64,S64>(ib, obj);
               default: fatal("AtomicSt: Operand type mismatch %d\n", ib->type);
             }
         } else {