[Mlir-commits] [mlir] 6451685 - Cleanup: Remove SmallVector hacks (#177667)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Jan 23 18:36:20 PST 2026


Author: AZero13
Date: 2026-01-23T18:36:15-08:00
New Revision: 6451685beae725d5b041df40c44668393c235ffa

URL: https://github.com/llvm/llvm-project/commit/6451685beae725d5b041df40c44668393c235ffa
DIFF: https://github.com/llvm/llvm-project/commit/6451685beae725d5b041df40c44668393c235ffa.diff

LOG: Cleanup: Remove SmallVector hacks (#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.

Note: I do not have merge permissions.

Added: 
    

Modified: 
    mlir/include/mlir/Support/LLVM.h

Removed: 
    


################################################################################
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