diff options
| author | Joe Loser <joeloser@fastmail.com> | 2022-09-06 18:06:58 -0600 |
|---|---|---|
| committer | Joe Loser <joeloser@fastmail.com> | 2022-09-08 09:01:53 -0600 |
| commit | 5e96cea1db0623a833d5376c9ea2ce4528771f97 (patch) | |
| tree | 7949b48788ad2e39d81d601464cdd2bb6f5c9e20 /llvm/unittests/Support/TargetParserTest.cpp | |
| parent | 7aec9ddcfd20dc241a37f862b20dddbb8a4efb2f (diff) | |
| download | llvm-5e96cea1db0623a833d5376c9ea2ce4528771f97.zip llvm-5e96cea1db0623a833d5376c9ea2ce4528771f97.tar.gz llvm-5e96cea1db0623a833d5376c9ea2ce4528771f97.tar.bz2 | |
[llvm] Use std::size instead of llvm::array_lengthof
LLVM contains a helpful function for getting the size of a C-style
array: `llvm::array_lengthof`. This is useful prior to C++17, but not as
helpful for C++17 or later: `std::size` already has support for C-style
arrays.
Change call sites to use `std::size` instead.
Differential Revision: https://reviews.llvm.org/D133429
Diffstat (limited to 'llvm/unittests/Support/TargetParserTest.cpp')
| -rw-r--r-- | llvm/unittests/Support/TargetParserTest.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/llvm/unittests/Support/TargetParserTest.cpp b/llvm/unittests/Support/TargetParserTest.cpp index 8d8327e..883600c6 100644 --- a/llvm/unittests/Support/TargetParserTest.cpp +++ b/llvm/unittests/Support/TargetParserTest.cpp @@ -736,7 +736,7 @@ TEST(TargetParserTest, ARMArchExtFeature) { {"mve", "nomve", "+mve", "-mve"}, {"mve.fp", "nomve.fp", "+mve.fp", "-mve.fp"}}; - for (unsigned i = 0; i < array_lengthof(ArchExt); i++) { + for (unsigned i = 0; i < std::size(ArchExt); i++) { EXPECT_EQ(StringRef(ArchExt[i][2]), ARM::getArchExtFeature(ArchExt[i][0])); EXPECT_EQ(StringRef(ArchExt[i][3]), ARM::getArchExtFeature(ArchExt[i][1])); } @@ -767,7 +767,7 @@ TEST(TargetParserTest, ARMArchExtDependencies) { TEST(TargetParserTest, ARMparseHWDiv) { const char *hwdiv[] = {"thumb", "arm", "arm,thumb", "thumb,arm"}; - for (unsigned i = 0; i < array_lengthof(hwdiv); i++) + for (unsigned i = 0; i < std::size(hwdiv); i++) EXPECT_NE(ARM::AEK_INVALID, ARM::parseHWDiv((StringRef)hwdiv[i])); } @@ -785,7 +785,7 @@ TEST(TargetParserTest, ARMparseArchEndianAndISA) { "v8.7a", "v8.8-a", "v8.8a", "v8-r", "v8m.base", "v8m.main", "v8.1m.main"}; - for (unsigned i = 0; i < array_lengthof(Arch); i++) { + for (unsigned i = 0; i < std::size(Arch); i++) { std::string arm_1 = "armeb" + (std::string)(Arch[i]); std::string arm_2 = "arm" + (std::string)(Arch[i]) + "eb"; std::string arm_3 = "arm" + (std::string)(Arch[i]); @@ -824,7 +824,7 @@ TEST(TargetParserTest, ARMparseArchEndianAndISA) { } TEST(TargetParserTest, ARMparseArchProfile) { - for (unsigned i = 0; i < array_lengthof(ARMArch); i++) { + for (unsigned i = 0; i < std::size(ARMArch); i++) { switch (ARM::parseArch(ARMArch[i])) { case ARM::ArchKind::ARMV6M: case ARM::ArchKind::ARMV7M: @@ -864,7 +864,7 @@ TEST(TargetParserTest, ARMparseArchProfile) { } TEST(TargetParserTest, ARMparseArchVersion) { - for (unsigned i = 0; i < array_lengthof(ARMArch); i++) + for (unsigned i = 0; i < std::size(ARMArch); i++) if (((std::string)ARMArch[i]).substr(0, 4) == "armv") EXPECT_EQ((ARMArch[i][4] - 48u), ARM::parseArchVersion(ARMArch[i])); else @@ -1585,7 +1585,7 @@ TEST(TargetParserTest, AArch64ArchExtFeature) { {"pmuv3", "nopmuv3", "+perfmon", "-perfmon"}, }; - for (unsigned i = 0; i < array_lengthof(ArchExt); i++) { + for (unsigned i = 0; i < std::size(ArchExt); i++) { EXPECT_EQ(StringRef(ArchExt[i][2]), AArch64::getArchExtFeature(ArchExt[i][0])); EXPECT_EQ(StringRef(ArchExt[i][3]), |
