-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59155/
-----------------------------------------------------------
(Updated May 10, 2017, 8:59 p.m.)
Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li.
Bugs: MESOS-7496
https://issues.apache.org/jira/browse/MESOS-7496
Repository: mesos
Description (updated)
-------
A previous commit attempted to enable this option, but it was not
actually being used by the linker. This patch correctly adds the option
to each linker flag variable for shared libraries, modules, and
executables, and only for the debug configuration. This results in much
faster link times.
Also, the `/zc:inline` option was removed, as it is only relevant in
optimized builds (i.e. release configurations).
Diffs (updated)
-----
cmake/CompilationConfigure.cmake 7b2669f0c54abf9b5e0fd60f8af01e97e1f0f86a
Diff: https://reviews.apache.org/r/59155/diff/2/
Changes: https://reviews.apache.org/r/59155/diff/1-2/
Testing
-------
Built and tested on Windows with VS Code.
NOTE: With `/debug:fastlink` properly enabled, the current VS Code C++ debugger
does not work without a manual update. See [this
thread](https://github.com/Microsoft/vscode-cpptools/issues/695#issuecomment-299554289)
for the work-around.
Thanks,
Andrew Schwartzmeyer
