cpu: Add missing rename of vector registers in the O3 CPU
authorRekai Gonzalez-Alberquilla <rekai.gonzalezalberquilla@arm.com>
Tue, 18 Jul 2017 15:31:38 +0000 (16:31 +0100)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Wed, 19 Jul 2017 10:01:15 +0000 (10:01 +0000)
The introduction of a new vector register class broke rename in the O3
CPU due to an unhandled register class in
DefaultRename<Impl>::renameSrcRegs(). This patch fixes adds the
necessary handling to avoid a panic when the vector register file is
used.

Change-Id: Ie380ab35ec4a151db15402f25b25b58931ee0581
Reviewed-by: Giacomo Gabrielli <giacomo.gabrielli@arm.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/4140
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/cpu/o3/rename_impl.hh

index b9adcdff7e8429c35451645ea70e4e8715d3f93d..bc024f603bbe4e72b80540b2e8a0a5ef39005833 100644 (file)
@@ -1028,6 +1028,9 @@ DefaultRename<Impl>::renameSrcRegs(DynInstPtr &inst, ThreadID tid)
           case FloatRegClass:
             fpRenameLookups++;
             break;
+          case VecRegClass:
+            vecRenameLookups++;
+            break;
           case CCRegClass:
           case MiscRegClass:
             break;