Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39680 )

Change subject: arch-power: Delete unused register related constants.
......................................................................

arch-power: Delete unused register related constants.

Change-Id: I7b2dc3a9ce29f67d304a22ab15268390fc461e4e
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/39680
Reviewed-by: Boris Shingarov <[email protected]>
Maintainer: Bobby R. Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/arch/power/registers.hh
1 file changed, 1 insertion(+), 13 deletions(-)

Approvals:
  Boris Shingarov: Looks good to me, approved
  Bobby R. Bruce: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/arch/power/registers.hh b/src/arch/power/registers.hh
index a6d28a8..def55e3 100644
--- a/src/arch/power/registers.hh
+++ b/src/arch/power/registers.hh
@@ -66,11 +66,9 @@
 // and zero register, which doesn't actually exist but needs a number
 const int NumIntSpecialRegs = 9;
 const int NumFloatArchRegs = 32;
-const int NumFloatSpecialRegs = 0;
-const int NumInternalProcRegs = 0;

 const int NumIntRegs = NumIntArchRegs + NumIntSpecialRegs;
-const int NumFloatRegs = NumFloatArchRegs + NumFloatSpecialRegs;
+const int NumFloatRegs = NumFloatArchRegs;
 const int NumVecRegs = 1;  // Not applicable to Power
                            // (1 to prevent warnings)
 const int NumVecPredRegs = 1;  // Not applicable to Power
@@ -80,21 +78,11 @@

 // Semantically meaningful register indices
 const int ReturnValueReg = 3;
-const int ArgumentReg0 = 3;
-const int ArgumentReg1 = 4;
-const int ArgumentReg2 = 5;
-const int ArgumentReg3 = 6;
-const int ArgumentReg4 = 7;
-const int FramePointerReg = 31;
 const int StackPointerReg = 1;

 // There isn't one in Power, but we need to define one somewhere
 const int ZeroReg = NumIntRegs - 1;

-const int SyscallNumReg = 0;
-const int SyscallPseudoReturnReg = 3;
-const int SyscallSuccessReg = 3;
-
 enum MiscIntRegNums {
     INTREG_CR = NumIntArchRegs,
     INTREG_XER,

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

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I7b2dc3a9ce29f67d304a22ab15268390fc461e4e
Gerrit-Change-Number: 39680
Gerrit-PatchSet: 4
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Boris Shingarov <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to