sim-se: const for loader's loadSection param
authorBrandon Potter <brandon.potter@amd.com>
Tue, 28 Aug 2018 18:49:51 +0000 (14:49 -0400)
committerBrandon Potter <Brandon.Potter@amd.com>
Wed, 29 May 2019 21:49:57 +0000 (21:49 +0000)
The port proxy can be declared as a reference to a const proxy
rather than just a reference to a proxy.

Change-Id: I4640b0c5f33e2334c1e7630131f78607ced40a34
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/12301
Maintainer: Brandon Potter <Brandon.Potter@amd.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
src/base/loader/elf_object.cc
src/base/loader/elf_object.hh
src/base/loader/object_file.cc
src/base/loader/object_file.hh

index 761fd79f5be27aaeb82f8ba5121ee8eb0fbc7ab4..654cb33d5db9401bf4774f477a76405374db666f 100644 (file)
@@ -498,7 +498,8 @@ ElfObject::loadWeakSymbols(SymbolTable *symtab, Addr base, Addr offset,
 }
 
 bool
-ElfObject::loadSections(PortProxy& mem_proxy, Addr addr_mask, Addr offset)
+ElfObject::loadSections(const PortProxy& mem_proxy, Addr addr_mask,
+                        Addr offset)
 {
     if (!ObjectFile::loadSections(mem_proxy, addr_mask, offset))
         return false;
index d74ee8914ea1551f6feda4e08d29e834c501d947..9585684d4e25ee548c7f3114e5544e7afc113d13 100644 (file)
@@ -94,7 +94,7 @@ class ElfObject : public ObjectFile
   public:
     virtual ~ElfObject() {}
 
-    bool loadSections(PortProxy& mem_proxy, Addr addr_mask = maxAddr,
+    bool loadSections(const PortProxy& mem_proxy, Addr addr_mask = maxAddr,
                       Addr offset = 0) override;
 
     virtual bool loadAllSymbols(SymbolTable *symtab, Addr base = 0,
index 86278e44fa5797808e6e66860c36ea776829f585..ede8922e92df9c94724ecfdf1015caeb0a8e2e10 100644 (file)
@@ -73,8 +73,8 @@ ObjectFile::~ObjectFile()
 
 
 bool
-ObjectFile::loadSection(Section *sec, PortProxy& mem_proxy, Addr addr_mask,
-                        Addr offset)
+ObjectFile::loadSection(Section *sec, const PortProxy& mem_proxy,
+                        Addr addr_mask, Addr offset)
 {
     if (sec->size != 0) {
         Addr addr = (sec->baseAddr & addr_mask) + offset;
@@ -91,7 +91,8 @@ ObjectFile::loadSection(Section *sec, PortProxy& mem_proxy, Addr addr_mask,
 
 
 bool
-ObjectFile::loadSections(PortProxy& mem_proxy, Addr addr_mask, Addr offset)
+ObjectFile::loadSections(const PortProxy& mem_proxy, Addr addr_mask,
+                         Addr offset)
 {
     return (loadSection(&text, mem_proxy, addr_mask, offset)
             && loadSection(&data, mem_proxy, addr_mask, offset)
index 5da7e3cc48258271dca0f348db35ec32023f9cdd..bd9d6260aa82dd1b0fab12dfab6f01b571068c92 100644 (file)
@@ -88,7 +88,7 @@ class ObjectFile
 
     static const Addr maxAddr = std::numeric_limits<Addr>::max();
 
-    virtual bool loadSections(PortProxy& mem_proxy,
+    virtual bool loadSections(const PortProxy& mem_proxy,
                               Addr mask = maxAddr, Addr offset = 0);
 
     virtual bool loadAllSymbols(SymbolTable *symtab, Addr base = 0,
@@ -129,7 +129,7 @@ class ObjectFile
     Section data;
     Section bss;
 
-    bool loadSection(Section *sec, PortProxy& mem_proxy, Addr mask,
+    bool loadSection(Section *sec, const PortProxy& mem_proxy, Addr mask,
                      Addr offset = 0);
     void setGlobalPointer(Addr global_ptr) { globalPtr = global_ptr; }