arch-arm: Fix ArmISA namespace requirement for TME instructions
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Wed, 9 Sep 2020 09:33:58 +0000 (10:33 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Wed, 9 Sep 2020 15:57:30 +0000 (15:57 +0000)
This is needed after:

https://gem5-review.googlesource.com/c/public/gem5/+/34155

Change-Id: I8ef0b5ce9cd5ae5224331e1c9347fdd9e884a536
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/34235
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/arm/insts/misc64.hh
src/arch/arm/insts/tme64.cc
src/arch/arm/insts/tme64.hh
src/arch/arm/insts/tme64classic.cc
src/arch/arm/insts/tme64ruby.cc

index 7aedd55584cf6b58d945adf9d632b292b79a7f42..a07788236c45a100bddc5cc0de99d8b1bd74eaa5 100644 (file)
@@ -234,14 +234,14 @@ class MiscRegImplDefined64 : public MiscRegOp64
             Addr pc, const Loader::SymbolTable *symtab) const override;
 };
 
-class RegNone : public ArmStaticInst
+class RegNone : public ArmISA::ArmStaticInst
 {
   protected:
-    IntRegIndex dest;
+    ArmISA::IntRegIndex dest;
 
-    RegNone(const char *mnem, ExtMachInst _machInst,
-                     OpClass __opClass, IntRegIndex _dest) :
-        ArmStaticInst(mnem, _machInst, __opClass),
+    RegNone(const char *mnem, ArmISA::ExtMachInst _machInst,
+            OpClass __opClass, ArmISA::IntRegIndex _dest) :
+        ArmISA::ArmStaticInst(mnem, _machInst, __opClass),
         dest(_dest)
     {}
 
index da228c4c2651592f4b130ccb71915c8f563a2d25..30aff208e6b050b38f54e3daa29ac2c24b26591d 100644 (file)
@@ -40,6 +40,8 @@
 
  #include <sstream>
 
+using namespace ArmISA;
+
 namespace ArmISAInst {
 
 std::string
index dada664e1091b06a839120dc135f23b50d43b612..b75adc1e67c3eef982f8b1d91d62d2cb322160b9 100644 (file)
 
 namespace ArmISAInst {
 
-class MicroTmeOp : public MicroOp
+class MicroTmeOp : public ArmISA::MicroOp
 {
   protected:
-    MicroTmeOp(const char *mnem, ExtMachInst machInst, OpClass __opClass) :
-               MicroOp(mnem, machInst, __opClass)
+    MicroTmeOp(const char *mnem, ArmISA::ExtMachInst machInst,
+               OpClass __opClass)
+      : ArmISA::MicroOp(mnem, machInst, __opClass)
     {}
 };
 
 class MicroTmeBasic64 : public MicroTmeOp
 {
   protected:
-    MicroTmeBasic64(const char *mnem, ExtMachInst machInst,
+    MicroTmeBasic64(const char *mnem, ArmISA::ExtMachInst machInst,
                     OpClass __opClass) :
                     MicroTmeOp(mnem, machInst, __opClass)
     {}
@@ -64,30 +65,30 @@ class MicroTmeBasic64 : public MicroTmeOp
                                     const Loader::SymbolTable *symtab) const;
 };
 
-class TmeImmOp64 : public ArmStaticInst
+class TmeImmOp64 : public ArmISA::ArmStaticInst
 {
   protected:
     uint64_t imm;
 
-    TmeImmOp64(const char *mnem, ExtMachInst machInst,
-               OpClass __opClass, uint64_t _imm) :
-                 ArmStaticInst(mnem, machInst, __opClass),
-                 imm(_imm)
+    TmeImmOp64(const char *mnem, ArmISA::ExtMachInst machInst,
+               OpClass __opClass, uint64_t _imm)
+      : ArmISA::ArmStaticInst(mnem, machInst, __opClass),
+        imm(_imm)
     {}
 
     std::string generateDisassembly(Addr pc,
                                     const Loader::SymbolTable *symtab) const;
 };
 
-class TmeRegNone64 : public ArmStaticInst
+class TmeRegNone64 : public ArmISA::ArmStaticInst
 {
   protected:
-    IntRegIndex dest;
+    ArmISA::IntRegIndex dest;
 
-    TmeRegNone64(const char *mnem, ExtMachInst machInst,
-                 OpClass __opClass, IntRegIndex _dest) :
-                   ArmStaticInst(mnem, machInst, __opClass),
-                   dest(_dest)
+    TmeRegNone64(const char *mnem, ArmISA::ExtMachInst machInst,
+                 OpClass __opClass, ArmISA::IntRegIndex _dest)
+      : ArmISA::ArmStaticInst(mnem, machInst, __opClass),
+        dest(_dest)
     {}
 
     std::string generateDisassembly(Addr pc,
@@ -97,7 +98,7 @@ class TmeRegNone64 : public ArmStaticInst
 class Tstart64 : public TmeRegNone64
 {
   public:
-    Tstart64(ExtMachInst, IntRegIndex);
+    Tstart64(ArmISA::ExtMachInst, ArmISA::IntRegIndex);
 
     Fault execute(ExecContext *, Trace::InstRecord *) const;
     Fault initiateAcc(ExecContext *, Trace::InstRecord *) const;
@@ -107,7 +108,7 @@ class Tstart64 : public TmeRegNone64
 class Ttest64 : public TmeRegNone64
 {
   public:
-    Ttest64(ExtMachInst, IntRegIndex);
+    Ttest64(ArmISA::ExtMachInst, ArmISA::IntRegIndex);
 
     Fault execute(ExecContext *, Trace::InstRecord *) const;
 };
@@ -115,7 +116,7 @@ class Ttest64 : public TmeRegNone64
 class Tcancel64 : public TmeImmOp64
 {
   public:
-    Tcancel64(ExtMachInst, uint64_t);
+    Tcancel64(ArmISA::ExtMachInst, uint64_t);
 
     Fault execute(ExecContext *, Trace::InstRecord *) const;
     Fault initiateAcc(ExecContext *, Trace::InstRecord *) const;
@@ -125,7 +126,7 @@ class Tcancel64 : public TmeImmOp64
 class MicroTfence64 : public MicroTmeBasic64
 {
   public:
-    MicroTfence64(ExtMachInst);
+    MicroTfence64(ArmISA::ExtMachInst);
 
     Fault execute(ExecContext *, Trace::InstRecord *) const;
     Fault initiateAcc(ExecContext *, Trace::InstRecord *) const;
@@ -135,7 +136,7 @@ class MicroTfence64 : public MicroTmeBasic64
 class MicroTcommit64 : public MicroTmeBasic64
 {
   public:
-    MicroTcommit64(ExtMachInst);
+    MicroTcommit64(ArmISA::ExtMachInst);
 
     Fault execute(ExecContext *, Trace::InstRecord *) const;
     Fault initiateAcc(ExecContext *, Trace::InstRecord *) const;
@@ -143,16 +144,17 @@ class MicroTcommit64 : public MicroTmeBasic64
 };
 
 
-class MacroTmeOp : public PredMacroOp
+class MacroTmeOp : public ArmISA::PredMacroOp
 {
   protected:
-    MacroTmeOp(const char *mnem, ExtMachInst _machInst, OpClass __opClass);
+    MacroTmeOp(const char *mnem, ArmISA::ExtMachInst _machInst,
+               OpClass __opClass);
 };
 
 class Tcommit64 : public MacroTmeOp
 {
   public:
-    Tcommit64(ExtMachInst _machInst);
+    Tcommit64(ArmISA::ExtMachInst _machInst);
 };
 
 } // namespace
index 3ad6e0a5219273b383770f86dcb252faf7a68967..b8fb627b37498ca2b0764e71b5697fb8300aceb9 100644 (file)
@@ -38,6 +38,8 @@
 #include "arch/arm/faults.hh"
 #include "arch/arm/insts/tme64.hh"
 
+using namespace ArmISA;
+
 namespace ArmISAInst {
 
 Fault
index a28c4e468057dc41aa5f9f075d74538c9f54d068..99481ba436efcd3e2543cbfda07d70e675b0c8e4 100644 (file)
@@ -45,8 +45,9 @@
 #include "mem/packet_access.hh"
 #include "mem/request.hh"
 
-namespace ArmISAInst {
+using namespace ArmISA;
 
+namespace ArmISAInst {
 
 Fault
 Tstart64::initiateAcc(ExecContext *xc,