[PATCH] D27230: [libFuzzer] Diff 2 - Separate FuzzerDef.h in appropriate header files.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 09:14:15 PST 2016


zturner added inline comments.


================
Comment at: lib/Fuzzer/FuzzerCorpus.h:20
+#include "FuzzerIO.h"
+#include "FuzzerSHA1.h"
 #include "FuzzerRandom.h"
----------------
mpividori wrote:
> zturner wrote:
> > Did you run clang-format on this patch?  It's supposed to alphabetize headers, but these are not in alphabetical order here.
> @zturner, thanks for your comment. I haven't run clang-format. I think libFuzzer need many changes to be according to LLVM Coding Standards. If you agree, I will add all of that changes in a new diff.
I got offline confirmation from kcc@ that running clang-format is not necessary.  So you don't need to run clang-format


Repository:
  rL LLVM

https://reviews.llvm.org/D27230





More information about the llvm-commits mailing list