For whatever reason, records in MethodList don't contain a name, but in OneMethod they do<br><div class="gmail_quote"><div dir="ltr">On Thu, May 12, 2016 at 8:22 AM Adrian McCarthy <<a href="mailto:amccarth@google.com">amccarth@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth added inline comments.<br>
<br>
================<br>
Comment at: include/llvm/DebugInfo/CodeView/TypeRecord.h:1018<br>
@@ +1017,3 @@<br>
+      Methods.emplace_back(L->Type, MethKind, Options, Access, VFTableOffset,<br>
+                           StringRef());<br>
+    }<br>
----------------<br>
My code is a little out of date, so I can't easily look up OneMethodRecord.  What does the empty `StringRef()` argument represent?<br>
<br>
================<br>
Comment at: include/llvm/DebugInfo/CodeView/TypeRecords.def:23<br>
@@ -18,1 +22,3 @@<br>
+#ifndef KNOWN_TYPE<br>
+#define KNOWN_TYPE(ename, value, class_name, print_name) LEAF_TYPE(ename, value)<br>
 #endif<br>
----------------<br>
rnk wrote:<br>
> I liked TYPE_RECORD and MEMBER_RECORD better than KNOWN_TYPE and MEMBER_TYPE. MEMBER_TYPES aren't really types, they're records describing class members.<br>
+1 for the old names.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D20190" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20190</a><br>
<br>
<br>
<br>
</blockquote></div>