[PATCH] D51008: Enable -fsanitize=fuzzer and -fsanitize=fuzzer-no-link on Windows.

2018-08-20 Thread Jonathan Metzman via Phabricator via cfe-commits
metzman created this revision. Herald added a subscriber: cfe-commits. This allows -fsanitize=fuzzer and -fsanitize=fuzzer-no-link to be used on Windows. When either used, Clang will also use some default flags that are needed (such as incremental linking and debug). Repository: rC Clang ht

[PATCH] D51008: Enable -fsanitize=fuzzer and -fsanitize=fuzzer-no-link on Windows.

2018-08-20 Thread Jonathan Metzman via Phabricator via cfe-commits
metzman added a reviewer: morehouse. metzman added a comment. Matt could you please take a look at this? I'll add rnk as reviewer once you sign off, since he is a CODE_OWNER for Windows in Clang. Thanks! Repository: rC Clang https://reviews.llvm.org/D51008 __

[PATCH] D51008: Enable -fsanitize=fuzzer and -fsanitize=fuzzer-no-link on Windows.

2018-08-20 Thread Jonathan Metzman via Phabricator via cfe-commits
metzman added inline comments. Comment at: lib/Driver/ToolChains/MSVC.cpp:373 + TC.getCompilerRTArgString(Args, "fuzzer", false))); +CmdArgs.push_back(Args.MakeArgString("-debug")); +// Prevent the linker from padding sections we use for instr

[PATCH] D51008: Enable -fsanitize=fuzzer and -fsanitize=fuzzer-no-link on Windows.

2018-08-21 Thread Jonathan Metzman via Phabricator via cfe-commits
metzman abandoned this revision. metzman added a comment. Abandoning this revision since I think the libFuzzer on Windows changes would be easier to understand as part of one commit instead of three. New revision here Repository: rC Clang https://reviews.llv