diff options
author | Gabe Black <gabeblack@google.com> | 2018-03-23 17:39:32 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2018-03-26 22:34:44 +0000 |
commit | cdf3cc2b95137807672ef7b8057695a13ce1cc33 (patch) | |
tree | 5d25308a7f9ded9c31f91968cd4e81d07c8e7e6b /src/arch/riscv/insts | |
parent | 740619f5d394da3816ff3fe6389cd7eb6ac55b8f (diff) | |
download | gem5-cdf3cc2b95137807672ef7b8057695a13ce1cc33.tar.xz |
arch: Fix all override related warnings.
Clang has started(?) reporting override related warnings, something gcc
apparently did before, but was disabled in the SConstruct. Rather than
disable the warnings in for clang as well, this change fixes the
warnings. A future change will re-enable the warnings for gcc.
Change-Id: I3cc79e45749b2ae0f9bebb1acadc56a3d3a942da
Reviewed-on: https://gem5-review.googlesource.com/9343
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Gabe Black <gabeblack@google.com>
Diffstat (limited to 'src/arch/riscv/insts')
-rw-r--r-- | src/arch/riscv/insts/static_inst.hh | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/arch/riscv/insts/static_inst.hh b/src/arch/riscv/insts/static_inst.hh index 3f6d58654..8ceb98490 100644 --- a/src/arch/riscv/insts/static_inst.hh +++ b/src/arch/riscv/insts/static_inst.hh @@ -51,11 +51,8 @@ class RiscvStaticInst : public StaticInst protected: using StaticInst::StaticInst; - virtual std::string - generateDisassembly(Addr pc, const SymbolTable *symtab) const = 0; - public: - void advancePC(PCState &pc) const { pc.advance(); } + void advancePC(PCState &pc) const override { pc.advance(); } size_t asBytes(void *buf, size_t size) override @@ -120,7 +117,7 @@ class RiscvMicroInst : public RiscvStaticInst flags[IsMicroop] = true; } - void advancePC(PCState &pcState) const; + void advancePC(PCState &pcState) const override; }; } |