LLD fails to dead strip user32.dll dependencies from chrome_elf.dll |
||||
Issue description
This is why chrome_elf_unittests is failing on CrWinClangLLD tester.
If you build chrome with lld, build the "chrome_elf" target, and run dumpbin /dependents, you can see the difference.
Here are the MSVC-linker produced chrome_elf dependencies:
KERNEL32.dll
VERSION.dll
Vs. LLD:
KERNEL32.dll
USER32.dll
VERSION.dll
SHELL32.dll
PSAPI.DLL
ole32.dll
WS2_32.dll
Most of the code in Chrome's //base library that depends on shell32, psapi, user32, etc is removed by MSVC's /OPT:REF implementation. LLD should do the same.
,
Mar 22 2017
,
Apr 25 2017
,
Apr 25 2017
Rui, is this still on your radar?
,
May 12 2017
A user filed this as https://bugs.llvm.org/show_bug.cgi?id=32950
,
Jun 5 2017
Can someone close this? I believe it's fixed, but it seems I don't have a permission to close.
,
Jun 5 2017
Yep, here's a green build from June 4: https://build.chromium.org/p/chromium.fyi/builders/CrWinClangLLD%20tester/builds/1503 Thanks! |
||||
►
Sign in to add a comment |
||||
Comment 1 by ruiu@google.com
, Feb 7 2017