[gem5-dev] [S] Change in gem5/gem5[release-staging-v23-0]: arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext

2023-06-08 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/71398?usp=email )


Change subject: arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext
..

arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext

Change-Id: Id5c5ae5fa1901154cadeb0a4958703f3f15d491f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67295
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/71398
Maintainer: Bobby Bruce 
---
M src/cpu/minor/BaseMinorCPU.py
1 file changed, 28 insertions(+), 0 deletions(-)

Approvals:
  kokoro: Regressions pass
  Bobby Bruce: Looks good to me, approved
  Jason Lowe-Power: Looks good to me, approved




diff --git a/src/cpu/minor/BaseMinorCPU.py b/src/cpu/minor/BaseMinorCPU.py
index bd27b92..c20a310 100644
--- a/src/cpu/minor/BaseMinorCPU.py
+++ b/src/cpu/minor/BaseMinorCPU.py
@@ -250,6 +250,33 @@
 opLat = 1


+class MinorDefaultVecFU(MinorFU):
+opClasses = minorMakeOpClassSet(
+[
+"VectorUnitStrideLoad",
+"VectorUnitStrideStore",
+"VectorUnitStrideMaskLoad",
+"VectorUnitStrideMaskStore",
+"VectorStridedLoad",
+"VectorStridedStore",
+"VectorIndexedLoad",
+"VectorIndexedStore",
+"VectorUnitStrideFaultOnlyFirstLoad",
+"VectorWholeRegisterLoad",
+"VectorWholeRegisterStore",
+"VectorIntegerArith",
+"VectorFloatArith",
+"VectorFloatConvert",
+"VectorIntegerReduce",
+"VectorFloatReduce",
+"VectorMisc",
+"VectorIntegerExtension",
+"VectorConfig",
+]
+)
+opLat = 1
+
+
 class MinorDefaultFUPool(MinorFUPool):
 funcUnits = [
 MinorDefaultIntFU(),
@@ -260,6 +287,7 @@
 MinorDefaultPredFU(),
 MinorDefaultMemFU(),
 MinorDefaultMiscFU(),
+MinorDefaultVecFU(),
 ]



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


Gerrit-MessageType: merged
Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v23-0
Gerrit-Change-Id: Id5c5ae5fa1901154cadeb0a4958703f3f15d491f
Gerrit-Change-Number: 71398
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Zhengrong Wang 
Gerrit-Reviewer: kokoro 
Gerrit-CC: kokoro 
Gerrit-CC: 轩胡 
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org


[gem5-dev] [S] Change in gem5/gem5[release-staging-v23-0]: arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext

2023-06-08 Thread Bobby Bruce (Gerrit) via gem5-dev

Attention is currently required from: Jason Lowe-Power.

Hello kokoro, Jason Lowe-Power,

I'd like you to do a code review.
Please visit

https://gem5-review.googlesource.com/c/public/gem5/+/71398?usp=email

to review the following change.


Change subject: arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext
..

arch-riscv,cpu-minor: Add MinorDefaultVecFU for risc-v v-ext

Change-Id: Id5c5ae5fa1901154cadeb0a4958703f3f15d491f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67295
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/cpu/minor/BaseMinorCPU.py
1 file changed, 28 insertions(+), 0 deletions(-)



diff --git a/src/cpu/minor/BaseMinorCPU.py b/src/cpu/minor/BaseMinorCPU.py
index bd27b92..c20a310 100644
--- a/src/cpu/minor/BaseMinorCPU.py
+++ b/src/cpu/minor/BaseMinorCPU.py
@@ -250,6 +250,33 @@
 opLat = 1


+class MinorDefaultVecFU(MinorFU):
+opClasses = minorMakeOpClassSet(
+[
+"VectorUnitStrideLoad",
+"VectorUnitStrideStore",
+"VectorUnitStrideMaskLoad",
+"VectorUnitStrideMaskStore",
+"VectorStridedLoad",
+"VectorStridedStore",
+"VectorIndexedLoad",
+"VectorIndexedStore",
+"VectorUnitStrideFaultOnlyFirstLoad",
+"VectorWholeRegisterLoad",
+"VectorWholeRegisterStore",
+"VectorIntegerArith",
+"VectorFloatArith",
+"VectorFloatConvert",
+"VectorIntegerReduce",
+"VectorFloatReduce",
+"VectorMisc",
+"VectorIntegerExtension",
+"VectorConfig",
+]
+)
+opLat = 1
+
+
 class MinorDefaultFUPool(MinorFUPool):
 funcUnits = [
 MinorDefaultIntFU(),
@@ -260,6 +287,7 @@
 MinorDefaultPredFU(),
 MinorDefaultMemFU(),
 MinorDefaultMiscFU(),
+MinorDefaultVecFU(),
 ]



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


Gerrit-MessageType: newchange
Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v23-0
Gerrit-Change-Id: Id5c5ae5fa1901154cadeb0a4958703f3f15d491f
Gerrit-Change-Number: 71398
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-CC: 轩胡 
Gerrit-Attention: Jason Lowe-Power 
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org