mips,cpu: Get rid of the IsDpsOp StaticInst flag.
authorGabe Black <gabeblack@google.com>
Sun, 30 Aug 2020 08:46:31 +0000 (01:46 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 15 Sep 2020 20:35:59 +0000 (20:35 +0000)
This flag was set by MIPS for a few instructions, but didn't have an
accessor in StaticInst and was never used for anything.

Change-Id: I153cedde0d16cb1d78b2705bd7340ebfd10e4fb6
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33740
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/mips/isa/formats/dsp.isa
src/cpu/StaticInstFlags.py

index 9a6d6147eb804dc51542dc77514fbaa21d7ed484..12af2d61fe608ec25d4527a87f3c54773bf9a76d 100644 (file)
@@ -173,8 +173,6 @@ def format DspIntOp(code, *opt_flags) {{
 
     code = decl_code + code + write_code
 
-    opt_flags += ('IsDspOp',)
-
     iop = InstObjParams(name, Name, 'DspIntOp', code, opt_flags)
     header_output = BasicDeclare.subst(iop)
     decoder_output = BasicConstructor.subst(iop)
@@ -204,8 +202,6 @@ def format DspHiLoOp(code, *opt_flags) {{
 
     code = decl_code + fetch_code + code + write_code
 
-    opt_flags += ('IsDspOp',)
-
     iop = InstObjParams(name, Name, 'DspHiLoOp', code, opt_flags)
     header_output = BasicDeclare.subst(iop)
     decoder_output = BasicConstructor.subst(iop)
index 151074edcc7c3d8f34b7a136d83dc76039e11860..27ba013ed4a3f4473f3c0175a80b8da86e900f48 100644 (file)
@@ -103,8 +103,9 @@ class StaticInstFlags(Enum):
         # This flag doesn't do anything yet
         'IsMicroBranch',    # This microop branches within the microcode for
                             # a macroop
-        'IsDspOp',
+
         'IsSquashAfter',     # Squash all uncommitted state after executed
+
         # hardware transactional memory
         'IsHtmStart',       # Starts a HTM transaction
         'IsHtmStop',        # Stops (commits) a HTM transaction