sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
Thanks! ================ Comment at: llvm/include/llvm/Support/Error.h:464 +/// errs() << "Problem with division " +/// << toString(E) << "\n"; +/// } ---------------- this won't compile, you need std::move(E) ================ Comment at: llvm/include/llvm/Support/Error.h:467 +/// // use the result +/// outs() << "The answer is " << *Result << "\n"; +/// @endcode ---------------- this will crash in the error case, you need to `return` after the errs() log Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105014/new/ https://reviews.llvm.org/D105014 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits