Commit fcf445dc authored by Keith Randall's avatar Keith Randall

runtime: initialize itab.hash always

We weren't initializing this field for dynamically-generated itabs.
Turns out it doesn't matter, as any time we use this field we also
generate a static itab for the interface type / concrete type pair.
But we should initialize it anyway, just to be safe.

Performance on the benchmarks in CL 44339:
benchmark               old ns/op     new ns/op     delta
BenchmarkItabFew-12     1040585       26466         -97.46%
BenchmarkItabAll-12     228873499     4287696       -98.13%

Change-Id: I58ed2b31e6c98b584122bdaf844fee7268b58295
Reviewed-on: https://go-review.googlesource.com/44475Reviewed-by: 's avatarJosh Bleecher Snyder <josharian@gmail.com>
parent 04d6f982
...@@ -222,6 +222,7 @@ imethods: ...@@ -222,6 +222,7 @@ imethods:
m.fun[0] = 0 m.fun[0] = 0
return iname return iname
} }
m.hash = typ.hash
return "" return ""
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment