diff options
| author | Peter Collingbourne <peter@pcc.me.uk> | 2017-11-20 18:51:29 +0000 |
|---|---|---|
| committer | Peter Collingbourne <peter@pcc.me.uk> | 2017-11-20 18:51:29 +0000 |
| commit | d01571353dfadbe5fb16f76b94ea591cc45f5b85 (patch) | |
| tree | 935dde53458055c5cba3b14e139a82b9e8890be4 /llvm/tools/llvm-rc/ResourceScriptParser.cpp | |
| parent | 8f54ae15a0d3f822ac834776665bf5676a7d24f7 (diff) | |
| download | llvm-d01571353dfadbe5fb16f76b94ea591cc45f5b85.zip llvm-d01571353dfadbe5fb16f76b94ea591cc45f5b85.tar.gz llvm-d01571353dfadbe5fb16f76b94ea591cc45f5b85.tar.bz2 | |
COFF: Stop requiring comdat sections to have an external leader to participate in ICF.
This requirement was added in r254578 to fix pr25686. However, it
appears to have originated from a misdiagnosis of the problem: link.exe
refused to merge the two sections because they are non-executable,
not because they have internal leaders. If I set up a similar scenario
with functions instead of globals I see that link.exe merges them.
Differential Revision: https://reviews.llvm.org/D40236
llvm-svn: 318682
Diffstat (limited to 'llvm/tools/llvm-rc/ResourceScriptParser.cpp')
0 files changed, 0 insertions, 0 deletions