Skip to content

Commit 6134add

Browse files
author
metzman
committed
[libFuzzer][Windows] Use dllexport for all declarations in FuzzerInterface.h
Summary: Use dllexport for all declarations in FuzzerInterface.h Use it for clang even though clang supports default visibility attribute to prevent a warning from being thrown when LLVMFuzzerMutate is defined with dllexport. This makes `FUZZER_INTERFACE_VISIBILITY` (FuzzerInterface.h) consistent with `ATTRIBUTE_INTERFACE` (FuzzerDefs.h) when using clang on Windows. Reviewers: vitalybuka, morehouse Reviewed By: morehouse Differential Revision: https://reviews.llvm.org/D57305 git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk/lib/fuzzer@352395 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent ea14bf4 commit 6134add

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

FuzzerInterface.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ extern "C" {
2727

2828
// Define FUZZER_INTERFACE_VISIBILITY to set default visibility in a way that
2929
// doesn't break MSVC.
30-
#if defined(_MSC_VER) && !defined(__clang__)
30+
#if defined(_WIN32)
3131
#define FUZZER_INTERFACE_VISIBILITY __declspec(dllexport)
3232
#else
3333
#define FUZZER_INTERFACE_VISIBILITY __attribute__((visibility("default")))

0 commit comments

Comments
 (0)