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

 (

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one. )Change subject: arch,sparc: Get rid of the unused checkVecEnableFault mechanism.
......................................................................

arch,sparc: Get rid of the unused checkVecEnableFault mechanism.

The method was only defined in SPARC, and SPARC does not use the vector
register file.

Change-Id: I4112eadaecb1f1c6c6db12975bec0cd1fa245d6c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/48715
Tested-by: kokoro <[email protected]>
Reviewed-by: Boris Shingarov <[email protected]>
Maintainer: Gabe Black <[email protected]>
---
M src/arch/isa_parser/isa_parser.py
M src/arch/sparc/isa/base.isa
2 files changed, 16 insertions(+), 8 deletions(-)

Approvals:
  Boris Shingarov: Looks good to me, approved
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass




diff --git a/src/arch/isa_parser/isa_parser.py b/src/arch/isa_parser/isa_parser.py
index a66059c..e084045 100755
--- a/src/arch/isa_parser/isa_parser.py
+++ b/src/arch/isa_parser/isa_parser.py
@@ -464,8 +464,6 @@
         # function (which should be provided by isa_desc via a declare)
         if 'IsFloating' in self.flags:
             self.fp_enable_check = 'fault = checkFpEnableFault(xc);'
-        elif 'IsVector' in self.flags:
-            self.fp_enable_check = 'fault = checkVecEnableFault(xc);'
         else:
             self.fp_enable_check = ''

diff --git a/src/arch/sparc/isa/base.isa b/src/arch/sparc/isa/base.isa
index 5be3940..89e48f0 100644
--- a/src/arch/sparc/isa/base.isa
+++ b/src/arch/sparc/isa/base.isa
@@ -136,12 +136,6 @@
             return NoFault;
         }
     }
-
-    static inline Fault
-    checkVecEnableFault(ExecContext *xc)
-    {
-        return std::make_shared<VecDisabled>();
-    }
 }};



--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/48715
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: I4112eadaecb1f1c6c6db12975bec0cd1fa245d6c
Gerrit-Change-Number: 48715
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Boris Shingarov <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-CC: Bobby R. Bruce <[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