morehouse accepted this revision. morehouse added inline comments. This revision is now accepted and ready to land.
================ Comment at: tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:50 + errs() << "error: opt level must be between 0 and 3.\n"; + std::exit(0); + } ---------------- `exit(1)` will indicate an abnormal exit. (here and below) ================ Comment at: tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:64 + SMDiagnostic Err; + // Create a new Context + LLVMContext Context; ---------------- This comment is unnecessary. Clearly a new Context is being created on the next line. Repository: rC Clang https://reviews.llvm.org/D48106 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
