search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

LLD COFF /MAP option not supported #31582

Open Quuxplusone opened 7 years ago

Quuxplusone commented 7 years ago
Bugzilla Link PR32609
Status CONFIRMED
Importance P normal
Reported by Ignat (ignacio.slipknot@gmail.com)
Reported on 2017-04-10 15:45:22 -0700
Last modified on 2019-06-08 07:03:51 -0700
Version unspecified
Hardware PC Windows NT
CC llvm-bugs@lists.llvm.org, llvm@thecybershadow.net, r.sagitario@gmx.de, ruiu@google.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also

Hello, this option is not supported by LLD 4.0

I use a commercial product that reads .map files generated by Visual Studio's linker, however lld doesn't support this option (if it did they should also be generated in the same format).

I can't patch the commercial product so it would be great if this feature was added.

Quuxplusone commented 7 years ago

I think /map is used mostly for debugging, so we implemented /lldmap which emits similar information in a different format. Since the format used for /lldmap is the same as the LLD/ELF linker, I don't think we want to remove that.

Implementing MSVC-compatible /map option might make sense if it is easy to do, though.

Quuxplusone commented 6 years ago

As map files are mostly consumed by humans to look up some symbols addresses, having the exact same format is not critical in most cases.

Even if unsupported, it would be better to just emit a warning instead of an error for /MAP to allow replacing the MS linker without changing build scripts.

Quuxplusone commented 6 years ago

Rainer,

Thank you for the suggestion. I agree with you -- it makes sense to print out a help message to use /lldmap instead of /map if /map is given. I'll make a change.