[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