From 6dd60a6c1adb88f8f50f510ffa0520fc84a10dc6 Mon Sep 17 00:00:00 2001 From: "Bobby R. Bruce" Date: Mon, 8 May 2023 10:54:56 -0700 Subject: [PATCH] base,arch,mem: Remove {GE}M5_VAR_USED instances `[[maybe_unused]]` is to be used to specify that a variable is used. Change-Id: Ife2ac96111b3af13e182baba1f3456e48c3a9f9b Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/70397 Reviewed-by: Jason Lowe-Power Maintainer: Bobby Bruce Tested-by: kokoro Reviewed-by: Daniel Carvalho --- src/arch/arm/isa/insts/sme.isa | 18 +++++++++--------- src/base/gtest/serialization_fixture.hh | 4 ++-- src/base/inet.cc | 6 +++--- src/base/stats/group.cc | 3 ++- src/mem/ruby/system/Sequencer.cc | 2 +- 5 files changed, 17 insertions(+), 16 deletions(-) diff --git a/src/arch/arm/isa/insts/sme.isa b/src/arch/arm/isa/insts/sme.isa index b9f6115432..03a1b2c6e2 100644 --- a/src/arch/arm/isa/insts/sme.isa +++ b/src/arch/arm/isa/insts/sme.isa @@ -100,12 +100,12 @@ let {{ xc->tcBase()); uint8_t offset = imm & (0xf >> (findMsbSet(sizeof(TPElem)))); - M5_VAR_USED uint8_t tile_idx = + [[maybe_unused]] uint8_t tile_idx = imm >> (4 - findMsbSet(sizeof(TPElem))); - M5_VAR_USED uint8_t vec_idx = (WOp2 + offset) % eCount; + [[maybe_unused]] uint8_t vec_idx = (WOp2 + offset) % eCount; // Calculate the address - M5_VAR_USED Addr EA = XOp1 + XOp3 * sizeof(TPElem); + [[maybe_unused]] Addr EA = XOp1 + XOp3 * sizeof(TPElem); // Calculate the read predicate. One boolean per byte, // initialised to all true. @@ -164,10 +164,10 @@ let {{ unsigned eCount = ArmStaticInst::getCurSmeVecLen( xc->tcBase()); - M5_VAR_USED uint8_t vec_index = (WOp2 + imm) % eCount; + [[maybe_unused]] uint8_t vec_index = (WOp2 + imm) % eCount; // Calculate the address - M5_VAR_USED Addr EA = XOp1 + imm; + [[maybe_unused]] Addr EA = XOp1 + imm; ''' iop = InstObjParams(name, "Sme" + Name, "SmeLdrStrOp", @@ -378,12 +378,12 @@ let {{ xc->tcBase()); uint8_t offset = imm & (0xf >> (findMsbSet(sizeof(TPElem)))); - M5_VAR_USED uint8_t tile_idx = + [[maybe_unused]] uint8_t tile_idx = imm >> (4 - findMsbSet(sizeof(TPElem))); - M5_VAR_USED uint8_t vec_idx = (WOp2 + offset) % eCount; + [[maybe_unused]] uint8_t vec_idx = (WOp2 + offset) % eCount; // Calculate the address - M5_VAR_USED Addr EA = XOp1 + XOp3 * sizeof(TPElem); + [[maybe_unused]] Addr EA = XOp1 + XOp3 * sizeof(TPElem); // Calculate the write predicate. One boolean per byte, // initialised to all true. @@ -446,7 +446,7 @@ let {{ auto row = getTileHSlice(ZA, 0, vec_index); // Calculate the address - M5_VAR_USED Addr EA = XOp1 + imm; + [[maybe_unused]] Addr EA = XOp1 + imm; uint8_t data[MaxSmeVecLenInBytes]; diff --git a/src/base/gtest/serialization_fixture.hh b/src/base/gtest/serialization_fixture.hh index 65269e094c..9184e7f572 100644 --- a/src/base/gtest/serialization_fixture.hh +++ b/src/base/gtest/serialization_fixture.hh @@ -99,7 +99,7 @@ class SerializationFixture : public ::testing::Test { // Create the directory dirName = generateTempDirName(); - M5_VAR_USED int success = mkdir(dirName.c_str(), 0775); + [[maybe_unused]] int success = mkdir(dirName.c_str(), 0775); assert(!(success == -1 && errno != EEXIST)); } @@ -110,7 +110,7 @@ class SerializationFixture : public ::testing::Test // rmdir does not work std::remove(getCptPath().c_str()); // Remove the directory we created on SetUp - M5_VAR_USED int success = rmdir(dirName.c_str()); + [[maybe_unused]] int success = rmdir(dirName.c_str()); assert(success == 0); } }; diff --git a/src/base/inet.cc b/src/base/inet.cc index fc7505ecb7..24c8e7eb9e 100644 --- a/src/base/inet.cc +++ b/src/base/inet.cc @@ -301,7 +301,7 @@ Ip6Hdr::extensionLength() const const uint8_t *data = bytes() + IP6_HDR_LEN; uint8_t nxt = ip6_nxt; int len = 0; - GEM5_VAR_USED int all = plen(); + [[maybe_unused]] int all = plen(); while (ip6Extension(nxt)) { const Ip6Opt *ext = (const Ip6Opt *)data; @@ -324,7 +324,7 @@ Ip6Hdr::getExt(uint8_t ext_type) const const uint8_t *data = bytes() + IP6_HDR_LEN; uint8_t nxt = ip6_nxt; Ip6Opt* opt = NULL; - GEM5_VAR_USED int all = plen(); + [[maybe_unused]] int all = plen(); while (ip6Extension(nxt)) { opt = (Ip6Opt *)data; @@ -349,7 +349,7 @@ Ip6Hdr::proto() const { const uint8_t *data = bytes() + IP6_HDR_LEN; uint8_t nxt = ip6_nxt; - GEM5_VAR_USED int all = plen(); + [[maybe_unused]] int all = plen(); while (ip6Extension(nxt)) { const Ip6Opt *ext = (const Ip6Opt *)data; diff --git a/src/base/stats/group.cc b/src/base/stats/group.cc index 93e7183f0e..addfdf9c28 100644 --- a/src/base/stats/group.cc +++ b/src/base/stats/group.cc @@ -72,7 +72,8 @@ Group::regStats() for (auto &g : statGroups) { if (debug::Stats) { - M5_VAR_USED const Named *named = dynamic_cast(this); + [[maybe_unused]] const Named *named = \ + dynamic_cast(this); DPRINTF(Stats, "%s: regStats in group %s\n", named ? named->name() : "?", g.first); diff --git a/src/mem/ruby/system/Sequencer.cc b/src/mem/ruby/system/Sequencer.cc index 3b75619969..82fc19b57c 100644 --- a/src/mem/ruby/system/Sequencer.cc +++ b/src/mem/ruby/system/Sequencer.cc @@ -230,7 +230,7 @@ Sequencer::wakeup() Cycles current_time = curCycle(); // Check across all outstanding requests - GEM5_VAR_USED int total_outstanding = 0; + [[maybe_unused]] int total_outstanding = 0; for (const auto &table_entry : m_RequestTable) { for (const auto &seq_req : table_entry.second) {