[LLDB] Default implementation for pack indexing types.

[cor3ntin]

We do not handle these types yet but we add them to some switch statement to ensure CI build does not fail.

[llvmbot]

@llvm/pr-subscribers-lldb

Author: cor3ntin (cor3ntin)

Changes

We do not handle these types yet but we add them to some switch statement to ensure CI build does not fail.

---

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

1 Files Affected:

• (modified) lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp (+12)

diff --git a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
index 6f5ff105477be1..4ceeb624e08888 100644
--- a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
+++ b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
@@ -4189,6 +4189,10 @@ TypeSystemClang::GetTypeClass(lldb::opaque_compiler_type_t type) {
   case clang::Type::ConstantMatrix:
   case clang::Type::DependentSizedMatrix:
     break;
+
+  // We don't handle pack indexing yet
+  case clang::Type::PackIndexing:
+    break;
   }
   // We don't know hot to display this type...
   return lldb::eTypeClassOther;
@@ -5066,6 +5070,10 @@ lldb::Encoding TypeSystemClang::GetEncoding(lldb::opaque_compiler_type_t type,
   case clang::Type::ConstantMatrix:
   case clang::Type::DependentSizedMatrix:
     break;
+
+  // We don't handle pack indexing yet
+  case clang::Type::PackIndexing:
+    break;
   }
   count = 0;
   return lldb::eEncodingInvalid;
@@ -5221,6 +5229,10 @@ lldb::Format TypeSystemClang::GetFormat(lldb::opaque_compiler_type_t type) {
   case clang::Type::ConstantMatrix:
   case clang::Type::DependentSizedMatrix:
     break;
+
+  // We don't handle pack indexing yet
+  case clang::Type::PackIndexing:
+    break;
   }
   // We don't know hot to display this type...
   return lldb::eFormatBytes;

[cor3ntin]

@DamonFool Can you review that for me? Thanks!

[cor3ntin]

I am landing this as to fix the CI build quickly
Feel free to do a post-commit review!

[DamonFool]

@DamonFool Can you review that for me? Thanks!

ninja check-lldb passed on Linux/x64.
So I think it's fine.
Thanks for fixing it.