[PATCH] D27348: [libFuzzer] Diff 11 - Add required header for Windows.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 11:53:43 PST 2016


zturner added a comment.

I know we originally said to split changes up into smaller patches, but here is an example of where I think the patch is too small :)

Diff 11, 12, 13, and 14 can all be combined into 1 patch, for example.  Maybe it could be called "Clean up headers and file formatting of LibFuzzer files".  Kostya mentioned that the complexity of reviewing a patch is N^2 in the size of the patch, but when N < 1, N > N^2  :)

Can you abandoned 12, 13, and 14, combine them all into 1 patch, and re-upload it here?


Repository:
  rL LLVM

https://reviews.llvm.org/D27348





More information about the llvm-commits mailing list