[PATCH] D55228: [PDB] Emit S_UDT records in LLD

[Alexandre Ganea via Phabricator via llvm-commits]

aganea added inline comments.


================
Comment at: lld/test/COFF/s_udt.s:20
+# CHECK:      {{.*}} | S_UDT [size = 24] `StructTypedef`
+# CHECK:               original type = 0x1003
+# CHECK:      {{.*}} | S_UDT [size = 16] `Struct`
----------------
Just to be sure, 0x1003 is here because of the yet unlanded D55236? It should read 0x1006 once it lands? You would need to re-generate this test at that point?


================
Comment at: llvm/lib/DebugInfo/PDB/Native/GSIStreamBuilder.cpp:64
 
   template <typename T> void addSymbol(const T &Symbol, MSFBuilder &Msf) {
     T Copy(Symbol);
----------------
Since you removed `GSIStreamBuilder::addGlobalSymbol` below, do we still need this function?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55228/new/

https://reviews.llvm.org/D55228