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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to