[Lldb-commits] [lldb] [LLDB] Run MSVC variant test with PDB (PR #171858)

[Michael Buch via lldb-commits]

================
@@ -119,8 +119,14 @@ bool formatters::MsvcStlVariantSummaryProvider(
     storage_type = storage_type.GetTypedefedType();
 
   CompilerType active_type = storage_type.GetTypeTemplateArgument(1, true);
-  if (!active_type)
-    return false;
+  if (!active_type) {
+    // PDB: get the type from the head as we don't have template arguments
+    // there.
+    ValueObjectSP head = GetHead(*storage);
----------------
Michael137 wrote:

Do we need a `nullcheck` on `head` here?

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