mips,cpu: Get rid of the IsIprAccess StaticInst flag.
authorGabe Black <gabeblack@google.com>
Sun, 30 Aug 2020 08:38:47 +0000 (01:38 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 15 Sep 2020 08:03:26 +0000 (08:03 +0000)
commit9f887b7634e3ec299e2c0b8ac8b982f7c99f9622
tree66e1854cf8c8d91732085d1a2b865e7976b4d173
parentd64465c02482fc2afa4c1467650976856c9a440a
mips,cpu: Get rid of the IsIprAccess StaticInst flag.

This was set by MIPS in two places, I think largely just because it was
available. This flag refers to IPRs which are an Alpha concept. In the
O3 CPU, IsIprAccess was used as a possible indicator to determine if an
instruction IsSerializeBefore, but we've already got a flag for that. In
the minor CPU, which hasn't been made to work with MIPS as far as I
know, it was used in a condition but not mentioned in the comment
alongside the condition. I think there it was added for the sake of
Alpha.

This change eliminates that flag and removes it from the O3 and minor
CPUs. In the MIPS ISA description, the instructions that were marked as
IsIprAccess have now been marked as IsSerializeBefore since, if there
was a real reason for them to be marked as IsIprAccess, it would have
been to get it them to work in O3, and there IsSerializeBefore gets
equivalent behavior.

Change-Id: Ia874cde12fa70b998d3e638458f13d69798d40b7
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33739
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
src/arch/mips/isa/decoder.isa
src/cpu/StaticInstFlags.py
src/cpu/base_dyn_inst.hh
src/cpu/minor/execute.cc
src/cpu/o3/rename_impl.hh
src/cpu/static_inst.hh