[clang] acdfd6b - [clang][bytecode] Char<Signed> doesn't need a descriptor ctor (#203748)

via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 14 23:47:55 PDT 2026


Author: Timm Baeder
Date: 2026-06-15T08:47:51+02:00
New Revision: acdfd6b71519c75f6d8c1188742634ed4a077def

URL: https://github.com/llvm/llvm-project/commit/acdfd6b71519c75f6d8c1188742634ed4a077def
DIFF: https://github.com/llvm/llvm-project/commit/acdfd6b71519c75f6d8c1188742634ed4a077def.diff

LOG: [clang][bytecode] Char<Signed> doesn't need a descriptor ctor (#203748)

This was lost when adding Char for 8-bit data.

Added: 
    

Modified: 
    clang/lib/AST/ByteCode/Descriptor.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/ByteCode/Descriptor.cpp b/clang/lib/AST/ByteCode/Descriptor.cpp
index dfe539ca85e04..8c5bb1af869d1 100644
--- a/clang/lib/AST/ByteCode/Descriptor.cpp
+++ b/clang/lib/AST/ByteCode/Descriptor.cpp
@@ -23,8 +23,8 @@ using namespace clang;
 using namespace clang::interp;
 
 template <typename T> static constexpr bool needsCtor() {
-  if constexpr (std::is_same_v<T, Integral<8, true>> ||
-                std::is_same_v<T, Integral<8, false>> ||
+  if constexpr (std::is_same_v<T, Char<true>> ||
+                std::is_same_v<T, Char<false>> ||
                 std::is_same_v<T, Integral<16, true>> ||
                 std::is_same_v<T, Integral<16, false>> ||
                 std::is_same_v<T, Integral<32, true>> ||


        


More information about the cfe-commits mailing list