misc: Add missing overrides to appease clang
authorAndreas Hansson <andreas.hansson@arm.com>
Mon, 15 Feb 2016 08:40:32 +0000 (03:40 -0500)
committerAndreas Hansson <andreas.hansson@arm.com>
Mon, 15 Feb 2016 08:40:32 +0000 (03:40 -0500)
Since the last round of fixes a few new issues have snuck in. We
should consider switching the regression runs to clang.

src/arch/sparc/system.hh
src/arch/sparc/tlb.hh
src/cpu/kvm/base.hh
src/cpu/kvm/x86_cpu.hh
src/cpu/minor/lsq.hh
src/dev/net/etherswitch.hh
src/dev/x86/pc.hh
src/mem/ruby/system/WeightedLRUPolicy.hh

index 68a192cb94960a05f7d89de83bc9e88a9723ad16..d4b61bd9534b60e684c7e7f71fa96262572dc122 100644 (file)
@@ -48,7 +48,7 @@ class SparcSystem : public System
     SparcSystem(Params *p);
     ~SparcSystem();
 
-    virtual void initState();
+    void initState() override;
 
 /**
  * Serialization stuff
index cd4634ab8c0c31795c9f752a7a627451bbe40fa8..65b6aff72a82af31da36ed0b7b709b2df38a779e 100644 (file)
@@ -115,7 +115,7 @@ class TLB : public BaseTLB
             bool update_used = true);
 
     /** Remove all entries from the TLB */
-    void flushAll();
+    void flushAll() override;
 
   protected:
     /** Insert a PTE into the TLB. */
@@ -153,10 +153,10 @@ class TLB : public BaseTLB
     typedef SparcTLBParams Params;
     TLB(const Params *p);
 
-    void takeOverFrom(BaseTLB *otlb) {}
+    void takeOverFrom(BaseTLB *otlb) override {}
 
     void
-    demapPage(Addr vaddr, uint64_t asn)
+    demapPage(Addr vaddr, uint64_t asn) override
     {
         panic("demapPage(Addr) is not implemented.\n");
     }
index d57ac3421cdffb096733e1494fd6800fc380524b..4aeed22864bb5b1d8f96c24c176ae16c58651be3 100644 (file)
@@ -80,9 +80,9 @@ class BaseKvmCPU : public BaseCPU
     BaseKvmCPU(BaseKvmCPUParams *params);
     virtual ~BaseKvmCPU();
 
-    void init();
-    void startup();
-    void regStats();
+    void init() override;
+    void startup() override;
+    void regStats() override;
 
     void serializeThread(CheckpointOut &cp, ThreadID tid) const override;
     void unserializeThread(CheckpointIn &cp, ThreadID tid) override;
@@ -90,24 +90,24 @@ class BaseKvmCPU : public BaseCPU
     DrainState drain() override;
     void drainResume() override;
 
-    void switchOut();
-    void takeOverFrom(BaseCPU *cpu);
+    void switchOut() override;
+    void takeOverFrom(BaseCPU *cpu) override;
 
-    void verifyMemoryMode() const;
+    void verifyMemoryMode() const override;
 
-    MasterPort &getDataPort() { return dataPort; }
-    MasterPort &getInstPort() { return instPort; }
+    MasterPort &getDataPort() override { return dataPort; }
+    MasterPort &getInstPort() override { return instPort; }
 
     void wakeup(ThreadID tid = 0) override;
-    void activateContext(ThreadID thread_num);
-    void suspendContext(ThreadID thread_num);
+    void activateContext(ThreadID thread_num) override;
+    void suspendContext(ThreadID thread_num) override;
     void deallocateContext(ThreadID thread_num);
-    void haltContext(ThreadID thread_num);
+    void haltContext(ThreadID thread_num) override;
 
-    ThreadContext *getContext(int tn);
+    ThreadContext *getContext(int tn) override;
 
-    Counter totalInsts() const;
-    Counter totalOps() const;
+    Counter totalInsts() const override;
+    Counter totalOps() const override;
 
     /** Dump the internal state to the terminal. */
     virtual void dump() const;
index 14f16d54408be6615505f9af299bb760d9cf980c..7a1966965207c340d2c8f2cf310f90825a48adbc 100644 (file)
@@ -44,7 +44,7 @@ class X86KvmCPU : public BaseKvmCPU
     X86KvmCPU(X86KvmCPUParams *params);
     virtual ~X86KvmCPU();
 
-    void startup();
+    void startup() override;
 
     /** @{ */
     void dump() const override;
@@ -61,7 +61,7 @@ class X86KvmCPU : public BaseKvmCPU
   protected:
     typedef std::vector<struct kvm_msr_entry> KvmMSRVector;
 
-    Tick kvmRun(Tick ticks);
+    Tick kvmRun(Tick ticks) override;
 
     /**
      * Run the virtual CPU until draining completes.
@@ -78,12 +78,12 @@ class X86KvmCPU : public BaseKvmCPU
      *
      * @return Number of ticks executed
      */
-    Tick kvmRunDrain();
+    Tick kvmRunDrain() override;
 
     /** Wrapper that synchronizes state in kvm_run */
     Tick kvmRunWrapper(Tick ticks);
 
-    uint64_t getHostCycles() const;
+    uint64_t getHostCycles() const override;
 
     /**
      * Methods to access CPUID information using the extended
@@ -132,8 +132,8 @@ class X86KvmCPU : public BaseKvmCPU
     void setVCpuEvents(const struct kvm_vcpu_events &events);
     /** @} */
 
-    void updateKvmState();
-    void updateThreadContext();
+    void updateKvmState() override;
+    void updateThreadContext() override;
 
     /**
      * Inject pending interrupts from gem5 into the virtual CPU.
@@ -143,9 +143,9 @@ class X86KvmCPU : public BaseKvmCPU
     /**
      * Handle x86 legacy IO (in/out)
      */
-    Tick handleKvmExitIO();
+    Tick handleKvmExitIO() override;
 
-    Tick handleKvmExitIRQWindowOpen();
+    Tick handleKvmExitIRQWindowOpen() override;
 
     /**
      * Check if there are pending events in the vCPU that prevents it
@@ -158,7 +158,7 @@ class X86KvmCPU : public BaseKvmCPU
      * @return False if there are pending events in the guest, True
      * otherwise.
      */
-    bool archIsDrained() const;
+    bool archIsDrained() const override;
 
   private:
     /**
index 33d7c506b899763b91f3adc595902196ca5f9c79..09fb30d03b7c7e9c7506c7be70337cc2b37982c5 100644 (file)
@@ -98,17 +98,17 @@ class LSQ : public Named
         { }
 
       protected:
-        bool recvTimingResp(PacketPtr pkt)
+        bool recvTimingResp(PacketPtr pkt) override
         { return lsq.recvTimingResp(pkt); }
 
-        void recvReqRetry() { lsq.recvReqRetry(); }
+        void recvReqRetry() override { lsq.recvReqRetry(); }
 
         bool isSnooping() const override { return true; }
 
-        void recvTimingSnoopReq(PacketPtr pkt)
+        void recvTimingSnoopReq(PacketPtr pkt) override
         { return lsq.recvTimingSnoopReq(pkt); }
 
-        void recvFunctionalSnoop(PacketPtr pkt) { }
+        void recvFunctionalSnoop(PacketPtr pkt) override { }
     };
 
     DcachePort dcachePort;
index 69a3f40fd8a99ae756442fc2e6e5334175d5dd82..7603717779d91d032848bdb851a94c601766621c 100644 (file)
@@ -60,7 +60,7 @@ class EtherSwitch : public EtherObject
         return dynamic_cast<const Params*>(_params);
     }
 
-    EtherInt *getEthPort(const std::string &if_name, int idx);
+    EtherInt *getEthPort(const std::string &if_name, int idx) override;
 
   protected:
     /**
index 6cc57cb5d31be17a7555c884e23b99df1ae11e9f..fc209a67db562be4e74f8334351a55954c022d0f 100644 (file)
@@ -57,7 +57,7 @@ class Pc : public Platform
     /**
      * Do platform initialization stuff
      */
-    void init();
+    void init() override;
 
     Pc(const Params *p);
 
index 3150779b2bbef874d218e11db13b068021540157..3cee9cb473f5c76c6b22e66a95cac47e4d903307 100644 (file)
@@ -49,11 +49,11 @@ class WeightedLRUPolicy : public AbstractReplacementPolicy
     WeightedLRUPolicy(const Params* p);
     ~WeightedLRUPolicy();
 
-    void touch(int64_t set, int64_t way, Tick time);
+    void touch(int64_t set, int64_t way, Tick time) override;
     void touch(int64_t set, int64_t way, Tick time, int occupancy);
     int64_t getVictim(int64_t set) const override;
 
-    bool useOccupancy() const { return true; }
+    bool useOccupancy() const override { return true; }
 
     CacheMemory * m_cache;
     int **m_last_occ_ptr;