Index: COFF/InputFiles.cpp =================================================================== --- COFF/InputFiles.cpp +++ COFF/InputFiles.cpp @@ -288,6 +288,8 @@ if (SC && SC->getOutputCharacteristics() & IMAGE_SCN_MEM_EXECUTE) { StringRef Name; COFFObj->getSymbolName(Sym, Name); + if (getMachineType() == I386) + Name.consume_front("_"); PrevailingSectionMap[Name] = SectionNumber; } } @@ -297,9 +299,10 @@ const DenseMap &PrevailingSectionMap) { StringRef Name; COFFObj->getSymbolName(Sym, Name); - if (Name.consume_front(".pdata$") || Name.consume_front(".xdata$")) { - // For MinGW, treat .[px]data$ as implicitly associative to - // the symbol . + if (Name.consume_front(".pdata$") || Name.consume_front(".xdata$") || + Name.consume_front(".eh_frame$")) { + // For MinGW, treat .[px]data$ and .eh_frame$ as implicitly + // associative to the symbol . auto ParentSym = PrevailingSectionMap.find(Name); if (ParentSym != PrevailingSectionMap.end()) readAssociativeDefinition(Sym, Def, ParentSym->second); Index: test/COFF/associative-comdat-mingw-i386.s =================================================================== --- /dev/null +++ test/COFF/associative-comdat-mingw-i386.s @@ -0,0 +1,37 @@ +# REQUIRES: x86 + +# RUN: llvm-mc -triple=i686-windows-gnu %s -filetype=obj -o %t.obj + +# RUN: lld-link -lldmingw -entry:main %t.obj -out:%t.exe +# RUN: llvm-objdump -s %t.exe | FileCheck %s + +# Check that the .eh_frame comdat was included, even if it had no symbols, +# due to associativity with the symbol _foo. + +# CHECK: Contents of section .eh_fram: +# CHECK: 403000 42 + + .text + .def _main; + .scl 2; + .type 32; + .endef + .globl _main + .p2align 4, 0x90 +_main: + call _foo + ret + + .section .eh_frame$foo,"dr" + .linkonce discard + .byte 0x42 + + .def _foo; + .scl 2; + .type 32; + .endef + .section .text$foo,"xr",discard,foo + .globl _foo + .p2align 4 +_foo: + ret