[Mlir-commits] [llvm] [mlir] [LLVM][MLIR] Fix TU-local entity exposure for C++20 module compatibility (PR #198023)

[llvmlistbot at llvm.org]

================
@@ -25,15 +25,15 @@ using check_has_member_remove_if_t =
     decltype(std::declval<Set>().remove_if(std::declval<Fn>()));
 
 template <typename Set, typename Fn>
-static constexpr bool HasMemberRemoveIf =
+inline constexpr bool HasMemberRemoveIf =
----------------
RedNicStone wrote:

In C++17, constexpr at namespace scope has internal linkage (same as static), so just removing static would have no effect on TU-local leakage.

https://github.com/llvm/llvm-project/pull/198023