Hello Andreas Sandberg, Steve Reinhardt, Giacomo Travaglini,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21700
to look at the new patch set (#2).
Change subject: tests: Added GTests for base/bitfield.hh
......................................................................
tests: Added GTests for base/bitfield.hh
In addition to the tests, a more detailed explanation of how
"insertBits(..)" functions has been included in its doxygen
documentation. The previous explanation was ambigious and led to
confusion.
Change-Id: I2ae8608733ebaa8f8f726cbb3a2cd8639b69c6b7
---
M src/base/SConscript
M src/base/bitfield.hh
A src/base/bitfield.test.cc
3 files changed, 276 insertions(+), 1 deletion(-)
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/21700
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: I2ae8608733ebaa8f8f726cbb3a2cd8639b69c6b7
Gerrit-Change-Number: 21700
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Steve Reinhardt <ste...@gmail.com>
Gerrit-CC: Gabe Black <gabebl...@google.com>
Gerrit-MessageType: newpatchset
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
