Hello Anthony Gutierrez, Jason Lowe-Power, Giacomo Travaglini, Andreas Sandberg,

I'd like you to reexamine a change. Please visit

    https://gem5-review.googlesource.com/c/public/gem5/+/12685

to look at the new patch set (#5).

Change subject: dev, arm: fix error class-memaccess with GCC >= 8.1
......................................................................

dev, arm: fix error class-memaccess with GCC >= 8.1

From GCC 8.1 on GCC issues a warning when using memset et al on structs and
classes. With the way gem5 builds, this actually prevents successful
builds.

Instead of using a pointer with SCSIReply as type, we cast to a void
pointer to avoid the message. On the way we wrap the memset call into a
method of SCSIReply called reset for better code readability.

Signed-off-by: Maurice Becker <[email protected]>
Change-Id: I3ed3fd9714be5d253aba01ca00b1863e1ae5cb68
---
M src/dev/arm/ufs_device.cc
M src/dev/arm/ufs_device.hh
2 files changed, 5 insertions(+), 1 deletion(-)


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/12685
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: I3ed3fd9714be5d253aba01ca00b1863e1ae5cb68
Gerrit-Change-Number: 12685
Gerrit-PatchSet: 5
Gerrit-Owner: MadMaurice <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Anthony Gutierrez <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: MadMaurice <[email protected]>
Gerrit-MessageType: newpatchset
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to