replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[PECOFF] Enable dead-stripping even if /debug option is given. 

/debug makes MSVC link.exe to not remove unused sections from
the resulting executable. We did the same thing before. However,
I realized that the removal of associative section depends on
the dead-stripping pass in LLD, so we cannot disable that. Or
LLD may produce slightly broken executables that have too much
data in it (which could result in nasty subtle bugs).

This patch is a temporary measure to create correct executable.
Currently /debug does not have any real effect for LLD anyway.

I'll improve associative section handling in another patch, so that
they are removed from output without depending on the dead-stripping
pass.

llvm-svn: 222483
This commit is contained in:
Rui Ueyama 2014-11-20 23:59:40 +00:00
parent 01014cee76
commit 4a259cf4a5
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lld/lib/Driver/WinLinkDriver.cpp
View File

@ -1177,8 +1177,9 @@ bool WinLinkDriver::parse(int argc, const char *argv[],
case OPT_debug:
// LLD is not yet capable of creating a PDB file, so /debug does not have
// any effect, other than disabling dead stripping.
ctx.setDeadStripping(false);
// any effect.
// TODO: This should disable dead stripping. Currently we can't do that
// because removal of associative sections depends on dead stripping.
break;
case OPT_verbose:

2
lld/unittests/DriverTests/WinLinkDriverTest.cpp
View File

@ -547,7 +547,7 @@ TEST_F(WinLinkParserTest, SwapRunFromNet) {
TEST_F(WinLinkParserTest, Debug) {
EXPECT_TRUE(parse("link.exe", "/debug", "a.out", nullptr));
EXPECT_FALSE(_context.deadStrip());
EXPECT_TRUE(_context.deadStrip());
}
TEST_F(WinLinkParserTest, Fixed) {