[gem5-dev] Change in gem5/gem5[develop]: sim: Add missing overrides in the *Fault classes

[Nikos Nikoleris (Gerrit) via gem5-dev]

Nikos Nikoleris has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/33777 )

Change subject: sim: Add missing overrides in the *Fault classes
......................................................................

sim: Add missing overrides in the *Fault classes

Change-Id: I7a74df78f0f85ccf7fd896f98b301c1f998c1497
Signed-off-by: Nikos Nikoleris <nikos.nikole...@arm.com>
---
M src/sim/faults.hh
1 file changed, 9 insertions(+), 5 deletions(-)



diff --git a/src/sim/faults.hh b/src/sim/faults.hh
index e9a57c6..4b5c1a5 100644
--- a/src/sim/faults.hh
+++ b/src/sim/faults.hh
@@ -55,7 +55,11 @@
   public:
     UnimpFault(std::string _str) : panicStr(_str) {}

-    FaultName name() const { return "Unimplemented simulator feature"; }
+    FaultName
+    name() const override
+    {
+        return "Unimplemented simulator feature";
+    }
     void invoke(ThreadContext *tc, const StaticInstPtr &inst =
                 StaticInst::nullStaticInstPtr) override;
 };
@@ -63,7 +67,7 @@
 class ReExec : public FaultBase
 {
   public:
-    virtual FaultName name() const { return "Re-execution fault"; }
+    virtual FaultName name() const override { return "Re-execution fault";  
}
     void invoke(ThreadContext *tc, const StaticInstPtr &inst=
                 StaticInst::nullStaticInstPtr) override;
 };
@@ -78,7 +82,7 @@
 class SyscallRetryFault : public FaultBase
 {
   public:
-    virtual FaultName name() const { return "System call retry fault"; }
+    FaultName name() const override { return "System call retry fault"; }
     SyscallRetryFault() {}
     void invoke(ThreadContext *tc, const StaticInstPtr &inst=
                 StaticInst::nullStaticInstPtr) override;
@@ -89,7 +93,7 @@
   private:
     Addr vaddr;
   public:
-    FaultName name() const { return "Generic page table fault"; }
+    FaultName name() const override { return "Generic page table fault"; }
     GenericPageTableFault(Addr va) : vaddr(va) {}
     void invoke(ThreadContext *tc, const StaticInstPtr &inst=
                 StaticInst::nullStaticInstPtr) override;
@@ -101,7 +105,7 @@
   private:
     Addr vaddr;
   public:
-    FaultName name() const { return "Generic alignment fault"; }
+    FaultName name() const override { return "Generic alignment fault"; }
     GenericAlignmentFault(Addr va) : vaddr(va) {}
     void invoke(ThreadContext *tc, const StaticInstPtr &inst=
                 StaticInst::nullStaticInstPtr) override;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/33777
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: I7a74df78f0f85ccf7fd896f98b301c1f998c1497
Gerrit-Change-Number: 33777
Gerrit-PatchSet: 1
Gerrit-Owner: Nikos Nikoleris <nikos.nikole...@arm.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s