[PATCH] D47964: Modified protobuf and converter to add new signature, remove conditionals.

Matt Morehouse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 11 09:44:52 PDT 2018


morehouse added inline comments.


================
Comment at: tools/clang-fuzzer/cxx_loop_proto.proto:67
-    AssignmentStatement assignment = 1;
-    IfElse              ifelse     = 2;
-  }
----------------
Do you really want to get rid of if-else?


================
Comment at: tools/clang-fuzzer/proto-to-cxx/loop_proto_to_cxx.cpp:123
 std::ostream &operator<<(std::ostream &os, const LoopFunction &x) {
-  return os << "void foo(int *a, size_t s) {\n"
+  return os << "#include \"foo.h\"\n" // This line is just for testing
+            << "void foo(int *a, int *b, int *__restrict__ c, size_t s) {\n"
----------------
What's the purpose of this include?


Repository:
  rC Clang

https://reviews.llvm.org/D47964





More information about the cfe-commits mailing list