[Mlir-commits] [mlir] Cleanup: Remove SmallVector hacks (PR #177667)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Fri Jan 23 11:53:43 PST 2026
https://github.com/AZero13 created https://github.com/llvm/llvm-project/pull/177667
We no longer support any platform that uses Clang 5. This was a workaround for older clang versions where template arguments weren't merged between forward declarations and definitions correctly.
Since we don't support anything this old anymore, we can drop this workaround.
>From f72b6f24770af2f8249676565c114996e9d36a9e Mon Sep 17 00:00:00 2001
From: AZero13 <gfunni234 at gmail.com>
Date: Fri, 23 Jan 2026 14:53:27 -0500
Subject: [PATCH] Cleanup: Remove SmallVector hacks
We no longer support any platform that uses clang 5. This was a workaround for older clang versions where template arguments weren't merged between forward declarations and definitions correctly.
Since we don't support anything this old anymore, we can drop this workaround.
---
mlir/include/mlir/Support/LLVM.h | 8 --------
1 file changed, 8 deletions(-)
diff --git a/mlir/include/mlir/Support/LLVM.h b/mlir/include/mlir/Support/LLVM.h
index 020c0fba726c8..81bbd717c4f80 100644
--- a/mlir/include/mlir/Support/LLVM.h
+++ b/mlir/include/mlir/Support/LLVM.h
@@ -26,14 +26,6 @@
// it anyway.
#include "llvm/Support/LogicalResult.h"
-// Workaround for clang-5 (PR41549)
-#if defined(__clang_major__)
-#if __clang_major__ <= 5
-#include "llvm/ADT/DenseMapInfo.h"
-#include "llvm/ADT/SmallVector.h"
-#endif
-#endif
-
// Forward declarations.
namespace llvm {
// String types
More information about the Mlir-commits
mailing list