[clang] [clang][bytecode] Fix copy-paste error in needsCtor() (PR #215542)

via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 11 05:37:31 PDT 2026


llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

<details>
<summary>Changes</summary>

This was checking for `Integral<64, false>` twice.

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


1 Files Affected:

- (modified) clang/lib/AST/ByteCode/Descriptor.cpp (+2-2) 


``````````diff
diff --git a/clang/lib/AST/ByteCode/Descriptor.cpp b/clang/lib/AST/ByteCode/Descriptor.cpp
index 3d661455460e3..12920241b8a89 100644
--- a/clang/lib/AST/ByteCode/Descriptor.cpp
+++ b/clang/lib/AST/ByteCode/Descriptor.cpp
@@ -30,10 +30,10 @@ template <typename T> static constexpr bool needsCtor() {
                 std::is_same_v<T, Integral<32, true>> ||
                 std::is_same_v<T, Integral<32, false>> ||
                 std::is_same_v<T, Integral<64, true>> ||
+                std::is_same_v<T, Integral<64, true>> ||
                 std::is_same_v<T, Integral<64, false>> ||
-                std::is_same_v<T, Integral<64, false>> ||
-                std::is_same_v<T, IntegralAP<false>> ||
                 std::is_same_v<T, IntegralAP<true>> ||
+                std::is_same_v<T, IntegralAP<false>> ||
                 std::is_same_v<T, Floating> || std::is_same_v<T, Boolean>)
     return false;
 

``````````

</details>


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


More information about the cfe-commits mailing list