[llvm] [Orc] Migrate ExecutorAddr::rawPtr to llvm::identity_cxx20 (NFC) (PR #164364)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 20 22:59:14 PDT 2025


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/164364

This patch updates ExecutorAddr::rawPtr to use llvm::identity_cxx20 to
better align with C++20.

Note that llvm::identity is a templated struct, while
llvm::identity_cxx20 is a non-templated struct with a templated
operator().

This patch makes rawPtr a non-templated type alias and adjusts use
sites.


>From 89d4bd69fa9d0bca570081758b918432097d9505 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Mon, 20 Oct 2025 10:19:20 -0700
Subject: [PATCH] [Orc] Migrate ExecutorAddr::rawPtr to llvm::identity_cxx20
 (NFC)

This patch updates ExecutorAddr::rawPtr to use llvm::identity_cxx20 to
better align with C++20.

Note that llvm::identity is a templated struct, while
llvm::identity_cxx20 is a non-templated struct with a templated
operator().

This patch makes rawPtr a non-templated type alias and adjusts use
sites.
---
 .../llvm/ExecutionEngine/Orc/Shared/ExecutorAddress.h     | 8 ++++----
 .../llvm/ExecutionEngine/Orc/Shared/ExecutorSymbolDef.h   | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorAddress.h b/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorAddress.h
index a5f6c4f373a1e..4a32113b460e1 100644
--- a/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorAddress.h
+++ b/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorAddress.h
@@ -14,7 +14,7 @@
 #define LLVM_EXECUTIONENGINE_ORC_SHARED_EXECUTORADDRESS_H
 
 #include "llvm/ADT/DenseMapInfo.h"
-#include "llvm/ADT/identity.h"
+#include "llvm/ADT/STLForwardCompat.h"
 #include "llvm/ExecutionEngine/Orc/Shared/SimplePackedSerialization.h"
 #include "llvm/Support/FormatVariadic.h"
 #include "llvm/Support/raw_ostream.h"
@@ -34,7 +34,7 @@ using ExecutorAddrDiff = uint64_t;
 class ExecutorAddr {
 public:
   /// A wrap/unwrap function that leaves pointers unmodified.
-  template <typename T> using rawPtr = llvm::identity<T *>;
+  using rawPtr = llvm::identity_cxx20;
 
 #if __has_feature(ptrauth_calls)
   template <typename T> class PtrauthSignDefault {
@@ -63,10 +63,10 @@ class ExecutorAddr {
 #else
 
   /// Default wrap function to use on this host.
-  template <typename T> using defaultWrap = rawPtr<T>;
+  template <typename T> using defaultWrap = rawPtr;
 
   /// Default unwrap function to use on this host.
-  template <typename T> using defaultUnwrap = rawPtr<T>;
+  template <typename T> using defaultUnwrap = rawPtr;
 
 #endif
 
diff --git a/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorSymbolDef.h b/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorSymbolDef.h
index 0756ab5ea9881..9ad2934cb9aa0 100644
--- a/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorSymbolDef.h
+++ b/llvm/include/llvm/ExecutionEngine/Orc/Shared/ExecutorSymbolDef.h
@@ -33,8 +33,8 @@ class ExecutorSymbolDef {
     JITSymbolFlags Flags = BaseFlags;
     if (std::is_function_v<T>)
       Flags |= JITSymbolFlags::Callable;
-    return ExecutorSymbolDef(
-        ExecutorAddr::fromPtr(UP, ExecutorAddr::rawPtr<T>()), Flags);
+    return ExecutorSymbolDef(ExecutorAddr::fromPtr(UP, ExecutorAddr::rawPtr()),
+                             Flags);
   }
 
   /// Cast this ExecutorSymbolDef to a pointer of the given type.



More information about the llvm-commits mailing list