mem: Expose the raw packet accessor functions.
authorGabe Black <gabeblack@google.com>
Fri, 12 Oct 2018 11:53:00 +0000 (04:53 -0700)
committerGabe Black <gabeblack@google.com>
Fri, 12 Oct 2018 23:44:14 +0000 (23:44 +0000)
This avoids a place where data has its endianness switched so that when
the endianness based accessors switch it back it returns to normal. It
also makes it easier to show intent when accessing single bytes where
endianness doesn't matter, and there's no contextual endianness.

Change-Id: I1b97396c1b9bb39727d35112d90e3969e5fe0aab
Reviewed-on: https://gem5-review.googlesource.com/c/13455
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>

src/mem/packet.hh

index 0f45a7bae8fa7d38e5839fcd0a249b9389f6d413..515dcc701862678ef7276986ce040aa1e2f2fbf4 100644 (file)
@@ -1077,9 +1077,11 @@ class Packet : public Printable
     template <typename T>
     void set(T v, ByteOrder endian);
 
+#if THE_ISA != NULL_ISA
     /** Set the value in the data pointer to v as guest endian. */
     template <typename T>
     void set(T v);
+#endif
 
 
     /**
@@ -1173,7 +1175,6 @@ class Packet : public Printable
 
     /** @} */
 
-  private: // Private data accessor methods
     /** Get the data in the packet without byte swapping. */
     template <typename T>
     T getRaw() const;