[clang] [clang][bytecode][NFC] Remove FunctionPointer from PrimTypeVariant (PR #215465)

via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 10 23:18:40 PDT 2026


llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

<details>
<summary>Changes</summary>

FunctionPointers are always regular Pointers these days.

---
Full diff: https://github.com/llvm/llvm-project/pull/215465.diff


1 Files Affected:

- (modified) clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp (+4-5) 


``````````diff
diff --git a/clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp b/clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp
index f136301c02912..2529bfc6c1cbd 100644
--- a/clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp
+++ b/clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp
@@ -511,11 +511,10 @@ bool clang::interp::DoBitCastPtr(InterpState &S, CodePtr OpPC,
 }
 
 using PrimTypeVariant =
-    std::variant<Pointer, FunctionPointer, MemberPointer, FixedPoint,
-                 Char<false>, Char<true>, Integral<16, false>,
-                 Integral<16, true>, Integral<32, false>, Integral<32, true>,
-                 Integral<64, false>, Integral<64, true>, IntegralAP<true>,
-                 IntegralAP<false>, Boolean, Floating>;
+    std::variant<Pointer, MemberPointer, FixedPoint, Char<false>, Char<true>,
+                 Integral<16, false>, Integral<16, true>, Integral<32, false>,
+                 Integral<32, true>, Integral<64, false>, Integral<64, true>,
+                 IntegralAP<true>, IntegralAP<false>, Boolean, Floating>;
 
 // NB: This implementation isn't exactly ideal, but:
 //   1) We can't just do a bitcast here since we need to be able to

``````````

</details>


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


More information about the cfe-commits mailing list