Daniel Carvalho has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/67362?usp=email )
Change subject: base: Remove the Loader namespace
......................................................................
base: Remove the Loader namespace
This namespace has gone through the deprecation period
and can now be removed.
Change-Id: I87b763fccfcdf720909dfbda9c3fc8f6dea36a61
Signed-off-by: Daniel R. Carvalho <oda...@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67362
Tested-by: kokoro <noreply+kok...@google.com>
---
M src/arch/mips/process.hh
M src/arch/power/process.hh
M src/arch/riscv/process.hh
M src/base/loader/dtb_file.cc
M src/base/loader/dtb_file.hh
M src/base/loader/elf_object.cc
M src/base/loader/elf_object.hh
M src/base/loader/image_file.hh
M src/base/loader/image_file_data.cc
M src/base/loader/image_file_data.hh
M src/base/loader/memory_image.cc
M src/base/loader/memory_image.hh
M src/base/loader/object_file.cc
M src/base/loader/object_file.hh
M src/base/loader/raw_image.hh
M src/base/loader/symtab.cc
M src/base/loader/symtab.hh
M src/base/loader/symtab.test.cc
M src/cpu/profile.hh
M src/cpu/static_inst.hh
M src/sim/process.hh
21 files changed, 160 insertions(+), 165 deletions(-)
Approvals:
Daniel Carvalho: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/arch/mips/process.hh b/src/arch/mips/process.hh
index 181dd25..8b84ec1 100644
--- a/src/arch/mips/process.hh
+++ b/src/arch/mips/process.hh
@@ -34,7 +34,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class ObjectFile;
diff --git a/src/arch/power/process.hh b/src/arch/power/process.hh
index c8d8a79..9576c35 100644
--- a/src/arch/power/process.hh
+++ b/src/arch/power/process.hh
@@ -36,7 +36,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class ObjectFile;
diff --git a/src/arch/riscv/process.hh b/src/arch/riscv/process.hh
index ca0a050..64b9593 100644
--- a/src/arch/riscv/process.hh
+++ b/src/arch/riscv/process.hh
@@ -40,7 +40,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class ObjectFile;
diff --git a/src/base/loader/dtb_file.cc b/src/base/loader/dtb_file.cc
index 13e0264..f083b3e 100644
--- a/src/base/loader/dtb_file.cc
+++ b/src/base/loader/dtb_file.cc
@@ -40,7 +40,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/dtb_file.hh b/src/base/loader/dtb_file.hh
index c11b195..bed7cfc 100644
--- a/src/base/loader/dtb_file.hh
+++ b/src/base/loader/dtb_file.hh
@@ -35,7 +35,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/elf_object.cc b/src/base/loader/elf_object.cc
index dc2abb8..4b1467a 100644
--- a/src/base/loader/elf_object.cc
+++ b/src/base/loader/elf_object.cc
@@ -61,7 +61,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/elf_object.hh b/src/base/loader/elf_object.hh
index 6159b35..f084492 100644
--- a/src/base/loader/elf_object.hh
+++ b/src/base/loader/elf_object.hh
@@ -51,7 +51,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/image_file.hh b/src/base/loader/image_file.hh
index 194c956..f1d3955 100644
--- a/src/base/loader/image_file.hh
+++ b/src/base/loader/image_file.hh
@@ -39,7 +39,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/image_file_data.cc
b/src/base/loader/image_file_data.cc
index 57fb47f..525d577 100644
--- a/src/base/loader/image_file_data.cc
+++ b/src/base/loader/image_file_data.cc
@@ -42,7 +42,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/image_file_data.hh
b/src/base/loader/image_file_data.hh
index d02c499..4d1701d 100644
--- a/src/base/loader/image_file_data.hh
+++ b/src/base/loader/image_file_data.hh
@@ -38,7 +38,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/memory_image.cc
b/src/base/loader/memory_image.cc
index 5537f28..a3f378c 100644
--- a/src/base/loader/memory_image.cc
+++ b/src/base/loader/memory_image.cc
@@ -32,7 +32,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/memory_image.hh
b/src/base/loader/memory_image.hh
index 2c56f4c..1207e74 100644
--- a/src/base/loader/memory_image.hh
+++ b/src/base/loader/memory_image.hh
@@ -46,7 +46,6 @@
class PortProxy;
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/object_file.cc b/src/base/loader/object_file.cc
index 3aa5915..287f910 100644
--- a/src/base/loader/object_file.cc
+++ b/src/base/loader/object_file.cc
@@ -48,7 +48,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/object_file.hh b/src/base/loader/object_file.hh
index 0415bec..f078116 100644
--- a/src/base/loader/object_file.hh
+++ b/src/base/loader/object_file.hh
@@ -55,7 +55,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/raw_image.hh b/src/base/loader/raw_image.hh
index 7321ea4..29f4340 100644
--- a/src/base/loader/raw_image.hh
+++ b/src/base/loader/raw_image.hh
@@ -35,7 +35,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/symtab.cc b/src/base/loader/symtab.cc
index f2f54e9..941ea10 100644
--- a/src/base/loader/symtab.cc
+++ b/src/base/loader/symtab.cc
@@ -37,7 +37,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/symtab.hh b/src/base/loader/symtab.hh
index e48e400..2e50523 100644
--- a/src/base/loader/symtab.hh
+++ b/src/base/loader/symtab.hh
@@ -44,7 +44,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
diff --git a/src/base/loader/symtab.test.cc b/src/base/loader/symtab.test.cc
index e9edb11..3130553 100644
--- a/src/base/loader/symtab.test.cc
+++ b/src/base/loader/symtab.test.cc
@@ -48,7 +48,7 @@
* @return The error string, if any.
*/
std::string
-getSymbolError(const Loader::Symbol& symbol, const Loader::Symbol&
expected)
+getSymbolError(const loader::Symbol& symbol, const loader::Symbol&
expected)
{
std::stringstream ss;
@@ -83,7 +83,7 @@
*/
::testing::AssertionResult
checkSymbol(const char* m_symbol, const char* m_expected,
- const Loader::Symbol& symbol, const Loader::Symbol& expected)
+ const loader::Symbol& symbol, const loader::Symbol& expected)
{
const std::string error = getSymbolError(symbol, expected);
if (!error.empty()) {
@@ -101,8 +101,8 @@
* @return A GTest's assertion result, with error message on failure.
*/
::testing::AssertionResult
-checkTable(const Loader::SymbolTable& symtab,
- const std::initializer_list<Loader::Symbol>& expected)
+checkTable(const loader::SymbolTable& symtab,
+ const std::initializer_list<loader::Symbol>& expected)
{
if (expected.size() != (symtab.end() - symtab.begin())) {
return ::testing::AssertionFailure() << "the number of symbols in "
@@ -126,7 +126,7 @@
/** Test that the constructor creates an empty table. */
TEST(LoaderSymtabTest, EmptyConstruction)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
ASSERT_TRUE(symtab.empty());
ASSERT_TRUE(checkTable(symtab, {}));
}
@@ -134,9 +134,9 @@
/** Test that the insertion of a symbol with no name fails. */
TEST(LoaderSymtabTest, InsertSymbolNoName)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "", 0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "", 0x10};
ASSERT_FALSE(symtab.insert(symbol));
ASSERT_TRUE(checkTable(symtab, {}));
}
@@ -144,9 +144,9 @@
/** Test that the insertion of one symbol in an empty table works. */
TEST(LoaderSymtabTest, InsertOneSymbol)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
ASSERT_TRUE(symtab.insert(symbol));
ASSERT_FALSE(symtab.empty());
@@ -156,12 +156,12 @@
/** Test that the insertion of a symbol with an existing name fails. */
TEST(LoaderSymtabTest, InsertSymbolExistingName)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
const std::string name = "symbol";
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, name, 0x10},
- {Loader::Symbol::Binding::Local, name, 0x20},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, name, 0x10},
+ {loader::Symbol::Binding::Local, name, 0x20},
};
ASSERT_TRUE(symtab.insert(symbols[0]));
ASSERT_FALSE(symtab.insert(symbols[1]));
@@ -173,12 +173,12 @@
/** Test that the insertion of a symbol with an existing address works. */
TEST(LoaderSymtabTest, InsertSymbolExistingAddress)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
const Addr addr = 0x10;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", addr},
- {Loader::Symbol::Binding::Local, "symbol2", addr},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", addr},
+ {loader::Symbol::Binding::Local, "symbol2", addr},
};
ASSERT_TRUE(symtab.insert(symbols[0]));
ASSERT_TRUE(symtab.insert(symbols[1]));
@@ -190,12 +190,12 @@
/** Test that the insertion of one symbol in a non-empty table works. */
TEST(LoaderSymtabTest, InsertMultipleSymbols)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -209,12 +209,12 @@
*/
TEST(LoaderSymtabTest, ClearMultiple)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -231,12 +231,12 @@
*/
TEST(LoaderSymtabTest, Offset)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -249,7 +249,7 @@
ASSERT_TRUE(checkTable(symtab, {symbols[0], symbols[1], symbols[2]}));
// Check that the new table is offset
- Loader::Symbol expected_symbols[] = {
+ loader::Symbol expected_symbols[] = {
{symbols[0].binding, symbols[0].name, symbols[0].address + offset},
{symbols[1].binding, symbols[1].name, symbols[1].address + offset},
{symbols[2].binding, symbols[2].name, symbols[2].address + offset},
@@ -264,13 +264,13 @@
*/
TEST(LoaderSymtabTest, Mask)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x1310},
- {Loader::Symbol::Binding::Local, "symbol2", 0x2810},
- {Loader::Symbol::Binding::Local, "symbol3", 0x2920},
- {Loader::Symbol::Binding::Local, "symbol4", 0x3C20},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x1310},
+ {loader::Symbol::Binding::Local, "symbol2", 0x2810},
+ {loader::Symbol::Binding::Local, "symbol3", 0x2920},
+ {loader::Symbol::Binding::Local, "symbol4", 0x3C20},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -285,7 +285,7 @@
symbols[3]}));
// Check that the new table is masked
- Loader::Symbol expected_symbols[] = {
+ loader::Symbol expected_symbols[] = {
{symbols[0].binding, symbols[0].name, symbols[0].address & mask},
{symbols[1].binding, symbols[1].name, symbols[1].address & mask},
{symbols[2].binding, symbols[2].name, symbols[2].address & mask},
@@ -301,13 +301,13 @@
*/
TEST(LoaderSymtabTest, Rename)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Local, "symbol4", 0x40},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Local, "symbol4", 0x40},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -322,7 +322,7 @@
symbols[3]}));
// Check that the new table's symbols have been renamed
- Loader::Symbol expected_symbols[] = {
+ loader::Symbol expected_symbols[] = {
{symbols[0].binding, symbols[0].name + "_suffix",
symbols[0].address},
{symbols[1].binding, symbols[1].name + "_suffix",
symbols[1].address},
{symbols[2].binding, symbols[2].name + "_suffix",
symbols[2].address},
@@ -338,13 +338,13 @@
*/
TEST(LoaderSymtabTest, RenameNonUnique)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Local, "symbol4", 0x40},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Local, "symbol4", 0x40},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -365,7 +365,7 @@
// Check that the new table's symbols have been renamed, yet it does
not
// contain the symbols with duplicated names
- Loader::Symbol expected_symbols[] = {
+ loader::Symbol expected_symbols[] = {
{symbols[0].binding, "NonUniqueName", symbols[0].address},
{symbols[1].binding, symbols[1].name, symbols[1].address},
{symbols[3].binding, symbols[3].name, symbols[3].address},
@@ -380,14 +380,14 @@
*/
TEST(LoaderSymtabTest, Globals)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Global, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Weak, "symbol4", 0x40},
- {Loader::Symbol::Binding::Weak, "symbol5", 0x50}
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Global, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Weak, "symbol4", 0x40},
+ {loader::Symbol::Binding::Weak, "symbol5", 0x50}
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -411,14 +411,14 @@
*/
TEST(LoaderSymtabTest, Locals)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Global, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Weak, "symbol4", 0x40},
- {Loader::Symbol::Binding::Weak, "symbol5", 0x50}
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Global, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Weak, "symbol4", 0x40},
+ {loader::Symbol::Binding::Weak, "symbol5", 0x50}
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -442,14 +442,14 @@
*/
TEST(LoaderSymtabTest, Weaks)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Global, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Weak, "symbol4", 0x40},
- {Loader::Symbol::Binding::Weak, "symbol5", 0x50}
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Global, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Weak, "symbol4", 0x40},
+ {loader::Symbol::Binding::Weak, "symbol5", 0x50}
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -470,9 +470,9 @@
/** Test searching for a non-existent address. */
TEST(LoaderSymtabTest, FindNonExistentAddress)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
EXPECT_TRUE(symtab.insert(symbol));
ASSERT_EQ(symtab.find(0x0), symtab.end());
@@ -481,12 +481,12 @@
/** Test searching for a unique address. */
TEST(LoaderSymtabTest, FindUniqueAddress)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -502,13 +502,13 @@
*/
TEST(LoaderSymtabTest, FindNonUniqueAddress)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
const Addr addr = 0x20;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", addr},
- {Loader::Symbol::Binding::Local, "symbol3", addr},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", addr},
+ {loader::Symbol::Binding::Local, "symbol3", addr},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -522,9 +522,9 @@
/** Test searching for a non-existent name. */
TEST(LoaderSymtabTest, FindNonExistentName)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
EXPECT_TRUE(symtab.insert(symbol));
const auto it = symtab.find("symbol2");
@@ -534,12 +534,12 @@
/** Test searching for an existing name. */
TEST(LoaderSymtabTest, FindExistingName)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -553,11 +553,11 @@
/** Test searching for an existent address using findNearest. */
TEST(LoaderSymtabTest, FindNearestExact)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -573,9 +573,9 @@
*/
TEST(LoaderSymtabTest, FindNearestRound)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
EXPECT_TRUE(symtab.insert(symbol));
const auto it = symtab.findNearest(symbol.address + 0x1);
@@ -590,11 +590,11 @@
*/
TEST(LoaderSymtabTest, FindNearestRoundWithNext)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -613,9 +613,9 @@
*/
TEST(LoaderSymtabTest, FindNearestRoundWithNextNonExistent)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
EXPECT_TRUE(symtab.insert(symbol));
Addr next_addr;
@@ -631,9 +631,9 @@
*/
TEST(LoaderSymtabTest, FindNearestNonExistent)
{
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
- Loader::Symbol symbol = {Loader::Symbol::Binding::Local, "symbol",
0x10};
+ loader::Symbol symbol = {loader::Symbol::Binding::Local, "symbol",
0x10};
EXPECT_TRUE(symtab.insert(symbol));
const auto it = symtab.findNearest(symbol.address - 0x1);
@@ -647,23 +647,23 @@
TEST(LoaderSymtabTest, InsertTableConflicting)
{
const std::string name = "symbol";
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, name, 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Local, "symbol4", 0x40},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, name, 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Local, "symbol4", 0x40},
// Introduce name conflict
- {Loader::Symbol::Binding::Local, name, 0x50},
+ {loader::Symbol::Binding::Local, name, 0x50},
};
// Populate table 1
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
EXPECT_TRUE(symtab.insert(symbols[2]));
// Populate table 2
- Loader::SymbolTable symtab2;
+ loader::SymbolTable symtab2;
EXPECT_TRUE(symtab2.insert(symbols[3]));
EXPECT_TRUE(symtab2.insert(symbols[4]));
@@ -681,22 +681,22 @@
*/
TEST(LoaderSymtabTest, InsertTable)
{
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
- {Loader::Symbol::Binding::Local, "symbol4", 0x40},
- {Loader::Symbol::Binding::Local, "symbol5", 0x50},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
+ {loader::Symbol::Binding::Local, "symbol4", 0x40},
+ {loader::Symbol::Binding::Local, "symbol5", 0x50},
};
// Populate table 1
- Loader::SymbolTable symtab;
+ loader::SymbolTable symtab;
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
EXPECT_TRUE(symtab.insert(symbols[2]));
// Populate table 2
- Loader::SymbolTable symtab2;
+ loader::SymbolTable symtab2;
EXPECT_TRUE(symtab2.insert(symbols[3]));
EXPECT_TRUE(symtab2.insert(symbols[4]));
@@ -717,11 +717,11 @@
TEST_F(LoaderSymtabSerializationFixture, Serialization)
{
// Populate the table
- Loader::SymbolTable symtab;
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::SymbolTable symtab;
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
EXPECT_TRUE(symtab.insert(symbols[0]));
EXPECT_TRUE(symtab.insert(symbols[1]));
@@ -742,17 +742,17 @@
/** Test unserialization. */
TEST_F(LoaderSymtabSerializationFixture, Unserialization)
{
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Local, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Local, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
simulateSerialization("\n[Section1]\ntest.size=3\n"
"test.addr_0=16\ntest.symbol_0=symbol\ntest.binding_0=1\n"
"test.addr_1=32\ntest.symbol_1=symbol2\ntest.binding_1=1\n"
"test.addr_2=48\ntest.symbol_2=symbol3\ntest.binding_2=1\n");
- Loader::SymbolTable unserialized_symtab;
+ loader::SymbolTable unserialized_symtab;
CheckpointIn cp(getDirName());
Serializable::ScopedCheckpointSection scs(cp, "Section1");
unserialized_symtab.unserialize("test", cp);
@@ -770,17 +770,17 @@
*/
TEST_F(LoaderSymtabSerializationFixture, UnserializationMissingBinding)
{
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Global, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Global, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
simulateSerialization("\n[Section1]\ntest.size=3\n"
"test.addr_0=16\ntest.symbol_0=symbol\ntest.binding_0=1\n"
"test.addr_1=32\ntest.symbol_1=symbol2\n"
"test.addr_2=48\ntest.symbol_2=symbol3\ntest.binding_2=1\n");
- Loader::SymbolTable unserialized_symtab;
+ loader::SymbolTable unserialized_symtab;
CheckpointIn cp(getDirName());
Serializable::ScopedCheckpointSection scs(cp, "Section1");
@@ -800,22 +800,22 @@
TEST_F(LoaderSymtabSerializationFixture,
UnserializationMissingBindingChangeDefault)
{
- Loader::Symbol symbols[] = {
- {Loader::Symbol::Binding::Local, "symbol", 0x10},
- {Loader::Symbol::Binding::Weak, "symbol2", 0x20},
- {Loader::Symbol::Binding::Local, "symbol3", 0x30},
+ loader::Symbol symbols[] = {
+ {loader::Symbol::Binding::Local, "symbol", 0x10},
+ {loader::Symbol::Binding::Weak, "symbol2", 0x20},
+ {loader::Symbol::Binding::Local, "symbol3", 0x30},
};
simulateSerialization("\n[Section1]\ntest.size=3\n"
"test.addr_0=16\ntest.symbol_0=symbol\ntest.binding_0=1\n"
"test.addr_1=32\ntest.symbol_1=symbol2\n"
"test.addr_2=48\ntest.symbol_2=symbol3\ntest.binding_2=1\n");
- Loader::SymbolTable unserialized_symtab;
+ loader::SymbolTable unserialized_symtab;
CheckpointIn cp(getDirName());
Serializable::ScopedCheckpointSection scs(cp, "Section1");
unserialized_symtab.unserialize("test", cp,
- Loader::Symbol::Binding::Weak);
+ loader::Symbol::Binding::Weak);
// Make sure that the symbols in symtab are present in the
// unserialized table
diff --git a/src/cpu/profile.hh b/src/cpu/profile.hh
index a5e16d6..68283f5 100644
--- a/src/cpu/profile.hh
+++ b/src/cpu/profile.hh
@@ -43,7 +43,6 @@
class ThreadContext;
class FunctionProfile;
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class SymbolTable;
diff --git a/src/cpu/static_inst.hh b/src/cpu/static_inst.hh
index 21ce2aa..3ab7834 100644
--- a/src/cpu/static_inst.hh
+++ b/src/cpu/static_inst.hh
@@ -66,7 +66,6 @@
class ExecContext;
class ThreadContext;
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class SymbolTable;
diff --git a/src/sim/process.hh b/src/sim/process.hh
index cece212..d6d30ce 100644
--- a/src/sim/process.hh
+++ b/src/sim/process.hh
@@ -49,7 +49,6 @@
namespace gem5
{
-GEM5_DEPRECATED_NAMESPACE(Loader, loader);
namespace loader
{
class ObjectFile;
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/67362?usp=email
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: I87b763fccfcdf720909dfbda9c3fc8f6dea36a61
Gerrit-Change-Number: 67362
Gerrit-PatchSet: 3
Gerrit-Owner: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: Richard Cooper <richard.coo...@arm.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org