Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/13455

Change subject: mem: Expose the raw packet accessor functions.
......................................................................

mem: Expose the raw packet accessor functions.

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
---
M src/mem/packet.hh
1 file changed, 2 insertions(+), 1 deletion(-)



diff --git a/src/mem/packet.hh b/src/mem/packet.hh
index 0f45a7b..515dcc7 100644
--- a/src/mem/packet.hh
+++ b/src/mem/packet.hh
@@ -1077,9 +1077,11 @@
     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 @@

     /** @} */

-  private: // Private data accessor methods
     /** Get the data in the packet without byte swapping. */
     template <typename T>
     T getRaw() const;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/13455
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I1b97396c1b9bb39727d35112d90e3969e5fe0aab
Gerrit-Change-Number: 13455
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to