diff options
author | Rui Ueyama <ruiu@google.com> | 2013-11-06 04:30:55 +0000 |
---|---|---|
committer | Rui Ueyama <ruiu@google.com> | 2013-11-06 04:30:55 +0000 |
commit | d213a9ed3c41d9eb0f14ee51b109a0f54404d94b (patch) | |
tree | 1df24d664229794c31acc979b4de0cbc0d2ddb1c /llvm/lib/Support/DynamicLibrary.cpp | |
parent | 3d7b33f4bfe7b602b6ac7092aebcda318d8f9d44 (diff) | |
download | llvm-d213a9ed3c41d9eb0f14ee51b109a0f54404d94b.zip llvm-d213a9ed3c41d9eb0f14ee51b109a0f54404d94b.tar.gz llvm-d213a9ed3c41d9eb0f14ee51b109a0f54404d94b.tar.bz2 |
[PECOFF] Do not wrap the linker internal file with an archive file.
We wrapped the linker internal file with a virtual archive file, so that the
linker internal file was linked only when it's actually used. This was to avoid
__ImageBase being included to the resulting executable. __ImageBase used to
occupy four bytes when emitted to executable.
And then it turned out that the implementation of __ImageBase was wrong -- it
shouldn't have been a regular atom but an absolute atom. Absolute atoms point
to some memory location, but they don't occupy disk space themselves. So it
wouldn't increase executable size (except the symbol table.) That means that
it's OK to link the linker internal file unconditionally.
So this patch does that, removing the wrapper archive file. Doing this
simplifies the code.
llvm-svn: 194127
Diffstat (limited to 'llvm/lib/Support/DynamicLibrary.cpp')
0 files changed, 0 insertions, 0 deletions