@crawshaw is this related to your recent changes?

The only change I've made recently to type information is exporting more symbols when building plugins, which shouldn't have anything to do with this. Maybe you mean someone else's changes?

Looks like this is a side-effect of 9c066ba

CL https://golang.org/cl/33233 mentions this issue.

If I understand correctly, the issue isn't actually that you need DWARF type info for temporary variables, but you need the DWARF type info for map-related struct types, and we used to only emit those because they were referenced in temporary variables?

I.e., reverting to the previous behavior where we emit DWARF type info for temporary variables fixes the issue, but really we just need to ensure that we emit info for all of those map-related struct types regardless of whether they're used as temporaries?

It isn't just map types, if you look at the example type info for main.astruct isn't emitted either.

@aarzilli Ah, thanks. I misread the type name in the bug report and thought you were talking about map.bucket or something.

In delve to print the value of an interface variable we used to parse the value of their tab._type field to reconstruct the name of the type and then look that up in debug_info, how should we do this now?

@aarzilli I was re-reviewing the fix for this issue, and I'm wondering: what does delve do with types that are constructed at runtime using reflect APIs? E.g., how does it handle:

package main

import (
	"fmt"
	"reflect"
)

type astruct struct {
	a, b int
}

func main() {
	stringTyp := reflect.TypeOf("")
	astructTyp := reflect.TypeOf((*astruct)(nil)).Elem()

	var iface interface{} = reflect.MakeMap(reflect.MapOf(stringTyp, astructTyp)).Interface()
	var iface2 interface{} = reflect.MakeSlice(reflect.SliceOf(astructTyp), 0, 0).Interface()
	fmt.Println(iface, iface2)
}

@mdempsky they are not supported. I have considered writing code to read the runtime._type and turn it directly into a dwarf.Type (without going through debug_info) but:

a) I would need the specialized versions of runtime._type (i.e. runtime.slicetype, runtime.maptype, etc) to be exported to debug_info.
b) it's a lot of work for something that has minimal use, so I've been putting it near the bottom of the list of things I should do for delve

For (a) I was hoping to propose this change during 1.9 since they are useful for other stuff, (b) is somewhat more fundamental.