[flang-commits] [flang] [flang] Use std::uintptr_t instead of unsigned long for better cross-platform stability (PR #183913)

[via flang-commits]

================
@@ -253,31 +253,17 @@ constexpr TypeBuilderFunc getModel<void (*)(int)>() {
 }
 template <>
 constexpr TypeBuilderFunc
-getModel<void *(*)(void *, const void *, unsigned long)>() {
+getModel<void *(*)(void *, const void *, std::uintptr_t)>() {
----------------
dragon-archer wrote:

Add a new specialization do make the compile success, but as I have said at the begining, that will make 3 *essentially* the same specialization coexist, which is very redundant.

I'm not sure whether using `uintptr_t` or `size_t` is better, as I can't find what the function type is meant for.

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