search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

LLD COFF handles /merge incorrectly #35187

Open Quuxplusone opened 6 years ago

Quuxplusone commented 6 years ago
Bugzilla Link PR36214
Status NEW
Importance P normal
Reported by Ignat (ignacio.slipknot@gmail.com)
Reported on 2018-02-02 12:11:18 -0800
Last modified on 2018-02-07 10:50:05 -0800
Version unspecified
Hardware PC Windows NT
CC colden@amazon.com, llvm-bugs@lists.llvm.org, llvm@inglorion.net, rnk@google.com, ruiu@google.com, zturner@google.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also

The current implementation stops when it finds a single merge flag, however it should continue until the final section name is resolved.

This is needed because MSVC passes something like /merge:.CRT=.rdata so /merge:.rdata=othersection won't merge the things from .CRT section

Quuxplusone commented 6 years ago

We also still need to do the internal /merge:.xdata=.rdata at some point...