Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(146)

Unified Diff: tools/supalink/README

Issue 7792103: Install link wrapper in runhooks, and turn incremental on by default (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: more cases in link sim Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/supalink/README
diff --git a/tools/supalink/README b/tools/supalink/README
new file mode 100644
index 0000000000000000000000000000000000000000..bdc3aab404d12d7d5bb7daed030f581bacb2ffe7
--- /dev/null
+++ b/tools/supalink/README
@@ -0,0 +1,20 @@
+Linker shim that enables the use of "Use Library Dependency Inputs" on
+large exe/dlls via Visual Studio.
+
+That flag is required to enable useful incremental linking, however, with a
+large number of objects in components, the linker fails with:
+
+...RSP00002E45885644.rsp : fatal error LNK1170: line in command file contains 131071 or more characters
+
+This seems to be that the IDE team didn't talk to the linker team; the
+response file can handle long commands, just all the files can't be on
+*one* line which is what the IDE generates.
+
+So, this program simply replaces link.exe, fixes the response file, and
+then shells to the original linker. Ridiculous? Yes. Faster links? Yes.
+
+Compile/install with install_supalink.py. Run from cmd.exe as Administrator,
+and make sure to run vsvars32.bat first.
+
+
+Scott Graham <scottmg@chromium.org>
M-A Ruel 2011/09/07 15:39:43 Remove, we know who you are. :)

Powered by Google App Engine
This is Rietveld 408576698