[PATCH] D22884: [codeview] Emit vftable records

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 12:57:02 PDT 2016


rnk added inline comments.

================
Comment at: test/DebugInfo/COFF/vftables.ll:45
@@ +44,3 @@
+
+; CHECK:  VFTable ([[a_vftable:0x[A-Z0-9]+]]) {
+; CHECK:    TypeLeafKind: LF_VFTABLE (0x151D)
----------------
I asked Microsoft about these records, and it turns out that they are completely unused for debugging. They only exist to power link-time devirtualization. In fact, they are never included in the final linked PDB.

So, I'm going to revert most of this patch, and do something more minimal to get the VFPtr records in the field list.


https://reviews.llvm.org/D22884





More information about the llvm-commits mailing list