mem: Remove explict cast from memhelper.
authorAli Saidi <Ali.Saidi@ARM.com>
Fri, 24 Jan 2014 21:29:30 +0000 (15:29 -0600)
committerAli Saidi <Ali.Saidi@ARM.com>
Fri, 24 Jan 2014 21:29:30 +0000 (15:29 -0600)
Previously we were casting the result type to the the memory type which
is incorrect for things like dual-memory operations which still return a
single result.

src/arch/generic/memhelpers.hh

index f7bbfa2691e19025d8513f8e5a095c3b4ffce714..7ed6141d04f2f9683bc110b3136062ccc7959723 100644 (file)
@@ -1,4 +1,16 @@
 /*
+ * Copyright (c) 2013 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder.  You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
  * Copyright (c) 2011 Google
  * All rights reserved.
  *
@@ -32,6 +44,7 @@
 #define __ARCH_GENERIC_MEMHELPERS_HH__
 
 #include "base/types.hh"
+#include "mem/request.hh"
 #include "sim/byteswap.hh"
 #include "sim/fault_fwd.hh"
 #include "sim/insttracer.hh"
@@ -92,7 +105,10 @@ writeMemAtomic(XC *xc, Trace::InstRecord *traceData, const MemT &mem,
 {
     Fault fault = writeMemTiming(xc, traceData, mem, addr, flags, res);
     if (fault == NoFault && res != NULL) {
-        *res = TheISA::gtoh((MemT)*res);
+        if (flags & Request::MEM_SWAP || flags & Request::MEM_SWAP_COND)
+            *res = TheISA::gtoh((MemT)*res);
+        else
+            *res = TheISA::gtoh(*res);
     }
     return fault;
 }