[PATCH] D32498: [CodeView] omit forward references for unnamed structs and unions

Brock Wyma via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 25 11:04:22 PDT 2017


bwyma created this revision.

When a forward reference is emitted, Visual Studio uses the name to match the forward reference to the definition.  For unnamed structs and unions there is no name to perform this matching.  Instead, references to unnamed types are expected to refer to the complete type definition.

The changes to CodeViewDebug omit the forward declaration for unnamed types.
The test DebugInfo/COFF/unnamed.ll was added to verify the correct emission of unnamed types.
The test DebugInfo/COFF/bitfields.ll needed an update because the source contains an unnamed struct.


https://reviews.llvm.org/D32498

Files:
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
  llvm/test/DebugInfo/COFF/bitfields.ll
  llvm/test/DebugInfo/COFF/unnamed.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32498.96602.patch
Type: text/x-patch
Size: 14039 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170425/7b68f752/attachment.bin>


More information about the llvm-commits mailing list