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

Reply via email to