[PATCH] D64587: Use clang driver for libfuzzer tests on Windows
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 11 13:30:19 PDT 2019
rnk created this revision.
rnk added reviewers: morehouse, thakis.
Herald added projects: Sanitizers, LLVM.
Herald added a subscriber: Sanitizers.
There's no real reason to use clang-cl on Windows, the clang driver
works just as well. This fixes a test which uses the -O0 flag, which was
recently removed from clang-cl to match MSVC, which lacks this flag.
While I'm here, remove the explicit -std=c++11 flag. Previously, this
flag was necessary when the default C++ standard was C++98. Now that the
default is C++14, this is no longer necessary. It's problematic on
Windows, because the Visual C++ standard library relies on C++14
features, and attempting to compile it with C++11 results in errors.
Rather than adding logic to conditionally set the standard to C++11 only
on non-Win, this flag can be removed.
See http://lab.llvm.org:8011/builders/clang-x64-windows-msvc and
https://reviews.llvm.org/D64506.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D64587
Files:
compiler-rt/test/fuzzer/lit.cfg.py
compiler-rt/test/fuzzer/windows-opt-ref.test
Index: compiler-rt/test/fuzzer/windows-opt-ref.test
===================================================================
--- compiler-rt/test/fuzzer/windows-opt-ref.test
+++ compiler-rt/test/fuzzer/windows-opt-ref.test
@@ -1,7 +1,7 @@
REQUIRES: windows
// Verify that the linker eliminating unreferenced functions (/OPT:REF) does not
// strip sancov module constructor.
-RUN: %cpp_compiler %S/SimpleCmpTest.cpp -o %t-SimpleCmpTest /link /OPT:REF
+RUN: %cpp_compiler %S/SimpleCmpTest.cpp -o %t-SimpleCmpTest -Wl,-OPT:REF
RUN: not %run %t-SimpleCmpTest -seed=1 -runs=100000000 2>&1 | FileCheck %s
Index: compiler-rt/test/fuzzer/lit.cfg.py
===================================================================
--- compiler-rt/test/fuzzer/lit.cfg.py
+++ compiler-rt/test/fuzzer/lit.cfg.py
@@ -66,10 +66,8 @@
compiler_cmd = config.clang
extra_cmd = config.target_flags
- if is_cpp and 'windows-msvc' in config.target_triple:
- std_cmd = '--driver-mode=cl'
- elif is_cpp:
- std_cmd = '--driver-mode=g++ -std=c++11'
+ if is_cpp:
+ std_cmd = '--driver-mode=g++'
else:
std_cmd = ''
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64587.209314.patch
Type: text/x-patch
Size: 1108 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190711/3ccad7b0/attachment.bin>
More information about the llvm-commits
mailing list