[PATCH] D27234: [libFuzzer] Diff 4 - Split FuzzerUtil implementation for Posix and Windows.

[Zachary Turner via Phabricator via llvm-commits]

zturner added inline comments.


================
Comment at: lib/Fuzzer/FuzzerUtil.h:64
 
+#if LIBFUZZER_WINDOWS
+const void *memmem(const void *haystack, size_t haystacklen,
----------------
kcc wrote:
> No such ifdefs, please. 
> Can this be moved to the win-specific file? 
It is called directly from non platform specific code.  The alternative would be to add a function like `SearchMemory()`, and in Posix file it just calls memmem, and in Win32 it does its own algorithm.  Then update callsites to use `SearchMemory()` instead.


Repository:
  rL LLVM

https://reviews.llvm.org/D27234