[Mlir-commits] [llvm] [mlir] [mlir][amdgpu] Improve Chipset version utility (PR #106169)
Jakub Kuderski
llvmlistbot at llvm.org
Mon Aug 26 18:50:36 PDT 2024
https://github.com/kuhar updated https://github.com/llvm/llvm-project/pull/106169
>From cecb508647747cfbef9f59e9d2d162b1b9524319 Mon Sep 17 00:00:00 2001
From: Jakub Kuderski <jakub at nod-labs.com>
Date: Mon, 26 Aug 2024 21:42:30 -0400
Subject: [PATCH 1/4] [mlir][amdgpu] Improve Chipset version utility
* Fix an OOB access
* Add comparison operators
* Add documentation
* Add unit tests
---
.../mlir/Dialect/AMDGPU/Utils/Chipset.h | 34 ++++++++--
mlir/lib/Dialect/AMDGPU/Utils/Chipset.cpp | 16 +++--
.../Dialect/AMDGPU/AMDGPUUtilsTest.cpp | 62 +++++++++++++++++++
mlir/unittests/Dialect/AMDGPU/CMakeLists.txt | 7 +++
mlir/unittests/Dialect/CMakeLists.txt | 1 +
.../mlir/unittests/BUILD.bazel | 13 ++++
6 files changed, 121 insertions(+), 12 deletions(-)
create mode 100644 mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
create mode 100644 mlir/unittests/Dialect/AMDGPU/CMakeLists.txt
diff --git a/mlir/include/mlir/Dialect/AMDGPU/Utils/Chipset.h b/mlir/include/mlir/Dialect/AMDGPU/Utils/Chipset.h
index 38e0ebe68f943b..0e2708b1efae03 100644
--- a/mlir/include/mlir/Dialect/AMDGPU/Utils/Chipset.h
+++ b/mlir/include/mlir/Dialect/AMDGPU/Utils/Chipset.h
@@ -9,19 +9,41 @@
#define MLIR_DIALECT_AMDGPU_UTILS_CHIPSET_H_
#include "mlir/Support/LLVM.h"
+#include <utility>
-namespace mlir {
-namespace amdgpu {
+namespace mlir::amdgpu {
+
+/// Represents the amdgpu gfx chipset version, e.g., gfx90a, gfx942, gfx1103.
+/// Note that the leading digits form a decimal number, while the last two
+/// digits for a hexadecimal number. For example:
+/// gfx942 --> major = 9, minor = 0x42
+/// gfx90a --> major = 9, minor = 0xa
+/// gfx1103 --> major = 10, minor = 0x3
struct Chipset {
Chipset() = default;
Chipset(unsigned majorVersion, unsigned minorVersion)
: majorVersion(majorVersion), minorVersion(minorVersion){};
+
+ /// Parses the chipset version string and returns the chipset on success, and
+ /// failure otherwise.
static FailureOr<Chipset> parse(StringRef name);
- unsigned majorVersion = 0;
- unsigned minorVersion = 0;
+ friend bool operator==(const Chipset &lhs, const Chipset &rhs) {
+ return lhs.majorVersion == rhs.majorVersion &&
+ lhs.minorVersion == rhs.minorVersion;
+ }
+ friend bool operator!=(const Chipset &lhs, const Chipset &rhs) {
+ return !(lhs == rhs);
+ }
+ friend bool operator<(const Chipset &lhs, const Chipset &rhs) {
+ return std::make_pair(lhs.majorVersion, lhs.minorVersion) <
+ std::make_pair(rhs.majorVersion, rhs.minorVersion);
+ }
+
+ unsigned majorVersion = 0; // The major version (decimal).
+ unsigned minorVersion = 0; // The minor version (hexadecimal).
};
-} // end namespace amdgpu
-} // end namespace mlir
+
+} // namespace mlir::amdgpu
#endif
diff --git a/mlir/lib/Dialect/AMDGPU/Utils/Chipset.cpp b/mlir/lib/Dialect/AMDGPU/Utils/Chipset.cpp
index 2540e1fb86d87d..fd15879d7b7ea0 100644
--- a/mlir/lib/Dialect/AMDGPU/Utils/Chipset.cpp
+++ b/mlir/lib/Dialect/AMDGPU/Utils/Chipset.cpp
@@ -1,4 +1,4 @@
-//===- Chipset.cpp - AMDGPU Chipset version struct parsing -----------===//
+//===- Chipset.cpp - AMDGPU Chipset version struct parsing ----------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
@@ -7,18 +7,20 @@
//===----------------------------------------------------------------------===//
#include "mlir/Dialect/AMDGPU/Utils/Chipset.h"
-#include "mlir/Support/LLVM.h"
#include "llvm/ADT/StringRef.h"
-using namespace mlir;
-using namespace mlir::amdgpu;
+namespace mlir::amdgpu {
FailureOr<Chipset> Chipset::parse(StringRef name) {
- if (!name.starts_with("gfx"))
+ if (!name.consume_front("gfx"))
return failure();
+ if (name.size() < 3)
+ return failure();
+
unsigned major = 0;
unsigned minor = 0;
- StringRef majorRef = name.drop_front(3).drop_back(2);
+
+ StringRef majorRef = name.drop_back(2);
StringRef minorRef = name.take_back(2);
if (majorRef.getAsInteger(10, major))
return failure();
@@ -26,3 +28,5 @@ FailureOr<Chipset> Chipset::parse(StringRef name) {
return failure();
return Chipset(major, minor);
}
+
+} // namespace mlir::amdgpu
diff --git a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
new file mode 100644
index 00000000000000..ba34e984d5643a
--- /dev/null
+++ b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
@@ -0,0 +1,62 @@
+//===- AMDGPUUtilsTest.cpp - Unit tests for AMDGPU dialect utils -----------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "mlir/Dialect/AMDGPU/Utils/Chipset.h"
+#include "gtest/gtest.h"
+
+namespace mlir::amdgpu {
+namespace {
+
+TEST(ChipsetTest, Parsing) {
+ FailureOr<Chipset> chipset = Chipset::parse("gfx90a");
+ ASSERT_TRUE(succeeded(chipset));
+ EXPECT_EQ(chipset->majorVersion, 9u);
+ EXPECT_EQ(chipset->minorVersion, 0x0au);
+
+ chipset = Chipset::parse("gfx940");
+ ASSERT_TRUE(succeeded(chipset));
+ EXPECT_EQ(chipset->majorVersion, 9u);
+ EXPECT_EQ(chipset->minorVersion, 0x40u);
+
+ chipset = Chipset::parse("gfx1103");
+ ASSERT_TRUE(succeeded(chipset));
+ EXPECT_EQ(chipset->majorVersion, 11u);
+ EXPECT_EQ(chipset->minorVersion, 0x03u);
+
+ ASSERT_TRUE(succeeded(chipset));
+ EXPECT_EQ(chipset->majorVersion, 11u);
+ EXPECT_EQ(chipset->minorVersion, 0x03u);
+}
+
+TEST(ChipsetTest, ParsingInvalid) {
+ EXPECT_TRUE(failed(Chipset::parse("navi33")));
+ EXPECT_TRUE(failed(Chipset::parse("rdna2")));
+ EXPECT_TRUE(failed(Chipset::parse("sm_80")));
+ EXPECT_TRUE(failed(Chipset::parse("GFX940")));
+ EXPECT_TRUE(failed(Chipset::parse("Gfx940")));
+ EXPECT_TRUE(failed(Chipset::parse("gfx9")));
+ EXPECT_TRUE(failed(Chipset::parse("gfx_940")));
+ EXPECT_TRUE(failed(Chipset::parse("gfx940_")));
+ EXPECT_TRUE(failed(Chipset::parse("gfxmeow")));
+ EXPECT_TRUE(failed(Chipset::parse("gfx1fff")));
+}
+
+TEST(ChipsetTest, Comparison) {
+ EXPECT_EQ(Chipset(9, 0x40), Chipset(9, 0x40));
+ EXPECT_NE(Chipset(9, 0x40), Chipset(9, 0x42));
+ EXPECT_NE(Chipset(9, 0x00), Chipset(10, 0x00));
+
+ EXPECT_LT(Chipset(9, 0x00), Chipset(10, 0x00));
+ EXPECT_LT(Chipset(9, 0x0a), Chipset(9, 0x42));
+ EXPECT_FALSE(Chipset(9, 0x42) < Chipset(9, 0x42));
+ EXPECT_FALSE(Chipset(9, 0x42) < Chipset(9, 0x40));
+}
+
+
+} // namespace
+} // namespace mlir::amdgpu
diff --git a/mlir/unittests/Dialect/AMDGPU/CMakeLists.txt b/mlir/unittests/Dialect/AMDGPU/CMakeLists.txt
new file mode 100644
index 00000000000000..d9a699e96288e8
--- /dev/null
+++ b/mlir/unittests/Dialect/AMDGPU/CMakeLists.txt
@@ -0,0 +1,7 @@
+add_mlir_unittest(MLIRAMDGPUTests
+ AMDGPUUtilsTest.cpp
+)
+target_link_libraries(MLIRAMDGPUTests
+ PRIVATE
+ MLIRAMDGPUUtils
+)
diff --git a/mlir/unittests/Dialect/CMakeLists.txt b/mlir/unittests/Dialect/CMakeLists.txt
index 90a75d5a46ad90..a5d4c48546e650 100644
--- a/mlir/unittests/Dialect/CMakeLists.txt
+++ b/mlir/unittests/Dialect/CMakeLists.txt
@@ -6,6 +6,7 @@ target_link_libraries(MLIRDialectTests
MLIRIR
MLIRDialect)
+add_subdirectory(AMDGPU)
add_subdirectory(ArmSME)
add_subdirectory(Index)
add_subdirectory(LLVMIR)
diff --git a/utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel b/utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel
index 7172beb4de9a62..a55c6f50118dcf 100644
--- a/utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel
@@ -137,6 +137,19 @@ cc_test(
],
)
+cc_test(
+ name = "amdgpu_tests",
+ size = "small",
+ srcs = glob([
+ "Dialect/AMDGPU/*.cpp",
+ ]),
+ deps = [
+ "//mlir:AMDGPUUtils",
+ "//third-party/unittest:gtest",
+ "//third-party/unittest:gtest_main",
+ ],
+)
+
cc_test(
name = "memref_tests",
size = "small",
>From 778b64974b00f55b6034ae86368dc98334ea32d6 Mon Sep 17 00:00:00 2001
From: Jakub Kuderski <jakub at nod-labs.com>
Date: Mon, 26 Aug 2024 21:47:23 -0400
Subject: [PATCH 2/4] cleanup
---
mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp | 4 ----
1 file changed, 4 deletions(-)
diff --git a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
index ba34e984d5643a..d0d89f7fb8b33f 100644
--- a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
+++ b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
@@ -27,10 +27,6 @@ TEST(ChipsetTest, Parsing) {
ASSERT_TRUE(succeeded(chipset));
EXPECT_EQ(chipset->majorVersion, 11u);
EXPECT_EQ(chipset->minorVersion, 0x03u);
-
- ASSERT_TRUE(succeeded(chipset));
- EXPECT_EQ(chipset->majorVersion, 11u);
- EXPECT_EQ(chipset->minorVersion, 0x03u);
}
TEST(ChipsetTest, ParsingInvalid) {
>From e067bbc937f86657e063c858f199ed6e74d5d8a1 Mon Sep 17 00:00:00 2001
From: Jakub Kuderski <jakub at nod-labs.com>
Date: Mon, 26 Aug 2024 21:50:10 -0400
Subject: [PATCH 3/4] fix formatting
---
mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
index d0d89f7fb8b33f..2eb4976aa1d212 100644
--- a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
+++ b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
@@ -1,4 +1,4 @@
-//===- AMDGPUUtilsTest.cpp - Unit tests for AMDGPU dialect utils -----------===//
+//===- AMDGPUUtilsTest.cpp - Unit tests for AMDGPU dialect utils ----------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
>From 3afd7a55456ed28c66b2b4dcd68a5d0245316e25 Mon Sep 17 00:00:00 2001
From: Jakub Kuderski <jakub at nod-labs.com>
Date: Mon, 26 Aug 2024 21:50:24 -0400
Subject: [PATCH 4/4] fix formatting
---
mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp | 1 -
1 file changed, 1 deletion(-)
diff --git a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
index 2eb4976aa1d212..b08b6681235d3b 100644
--- a/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
+++ b/mlir/unittests/Dialect/AMDGPU/AMDGPUUtilsTest.cpp
@@ -53,6 +53,5 @@ TEST(ChipsetTest, Comparison) {
EXPECT_FALSE(Chipset(9, 0x42) < Chipset(9, 0x40));
}
-
} // namespace
} // namespace mlir::amdgpu
More information about the Mlir-commits
mailing list