tools: benchmark improvements + testing fix
build: less MSVC warnings

Enable "Treat warnings as errors"

Removed/Silenced:
4005 - about redefinition of `PLATFORM` symbol
4996 - For the visible future I don't think we should be worried about the change in semantics of  `GetVersionEx`
4291 - for the `new` operator that has non side effects in FSReqWrap ```void *node::FSReqWrap::operator new(size_t,char *)```
4267 - when passing an int64 as int, and truncation might happen (depends on linkage)
4244 - when passing an int64 as int, and truncation will happen
4530 - No exception semantics (leaking from MS STL xlocale)

align macros between dtrace and ETW removes warnings:
NODE\SRC\NODE_DTRACE.CC(180) : warning C4002: too many actual parameters for macro 'NODE_NET_SOCKET_READ'
NODE\SRC\NODE_DTRACE.CC(196) : warning C4002: too many actual parameters for macro 'NODE_NET_SOCKET_WRITE'

added `verbose` to vcbuild.bat

more verbose VCLinkerTool
src: just get it to work with new v8 (no debugger)
src: use old v8 code as debug-agent