[PATCH] D74359: [compiler-rt] FuzzedDataProvider: add ConsumeData method.

Jonathan Metzman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 11 13:55:09 PST 2020


metzman accepted this revision.
metzman added a comment.

still LGTM



================
Comment at: compiler-rt/include/fuzzer/FuzzedDataProvider.h:246
+  // fuzzing data.
+  size_t ConsumeData(void *destination, size_t num_bytes) {
+    num_bytes = std::min(num_bytes, remaining_bytes_);
----------------
Might be better not to use the same name "Consume" as other methods since this method is used a bit differently than other methods which return their value.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74359/new/

https://reviews.llvm.org/D74359





More information about the llvm-commits mailing list