diff options
| author | Gabe Black <gabeblack@google.com> | 2020-01-09 00:40:32 -0800 |
|---|---|---|
| committer | Gabe Black <gabeblack@google.com> | 2020-01-11 10:00:26 +0000 |
| commit | 230a4bfd9c5a65a3b12226de486e05b566c7c006 (patch) | |
| tree | a55f500f4e6ff682393a027a19e660358305f6f4 | |
| parent | a58e5181d813d7a394f64fc434f0be04c4f67e95 (diff) | |
| download | gem5-230a4bfd9c5a65a3b12226de486e05b566c7c006.tar.xz | |
arch: Make the generic micropc enabled PCState set nupc to 1.
The default constructor of the micropc enabled generic PCState class
set the next micropc to 0, when the non-default constructor and at
least the x86 initCPU utility function set it to 1. This makes more
sense since either the micropc doesn't matter as a concept if the
instruction isn't microcoded, or, unless redirected by a micropc
branch, you're going to want to execute the next microop and not just
repeat the first one.
Change-Id: I418ea986a071453563c4c8aad4fc4eb4f7beb641
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/24184
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
| -rw-r--r-- | src/arch/generic/types.hh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/arch/generic/types.hh b/src/arch/generic/types.hh index 7f9f93f42..64fef6413 100644 --- a/src/arch/generic/types.hh +++ b/src/arch/generic/types.hh @@ -222,7 +222,7 @@ class UPCState : public SimplePCState<MachInst> nupc(1); } - UPCState() : _upc(0), _nupc(0) {} + UPCState() : _upc(0), _nupc(1) {} UPCState(Addr val) : _upc(0), _nupc(0) { set(val); } bool |