[clang] [lldb] [clang]: support std::meta::info for primitive types (PR #190356)

[Balázs Kéri via cfe-commits]

================
@@ -10954,6 +10965,27 @@ ASTNodeImporter::ImportAPValue(const APValue &FromValue) {
     } else
       Result.setLValue(Base, Offset, APValue::NoLValuePath{},
                        FromValue.isNullPointer());
+    break;
+  }
+  case APValue::Reflection: {
+    switch (FromValue.getReflectionOperandKind()) {
+    case ReflectionKind::Null:
+      Result = APValue(ReflectionKind::Null, nullptr);
+      break;
+    case ReflectionKind::Type: {
+      const auto *FromTSI = static_cast<const TypeSourceInfo *>(
+          FromValue.getReflectionOpaqueOperand());
+      QualType ImpType = importChecked(Err, FromTSI->getType());
----------------
balazske wrote:

Is there reason why not `importChecked(Err, FromTSI)` is used? Code would be more simple, and this will import additionally the BeginLoc in `TypeSourceInfo` object (but according to FIXME the import of a `TypeSourceInfo` is not complete).

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