[PATCH] D26884: Change setDiagnosticsOutputFile to take a unique_ptr from a raw pointer (NFC)
Mehdi AMINI via llvm-commits
llvm-commits at lists.llvm.org
Sat Nov 19 10:33:01 PST 2016
mehdi_amini added a comment.
Pushed inadvertently with https://reviews.llvm.org/D26886, I'll address the review as post-commit.
================
Comment at: llvm/include/llvm/IR/LLVMContext.h:197
/// set, the handler is invoked for each diagnostic message.
- void setDiagnosticsOutputFile(yaml::Output *F);
+ void setDiagnosticsOutputFile(std::unique_ptr<yaml::Output> F);
----------------
malcolm.parsons wrote:
> Pass by value if always moving doesn't apply to move-only types. Use && to avoid a temporary.
Sorry I don't understand this comment?
Repository:
rL LLVM
https://reviews.llvm.org/D26884
More information about the llvm-commits
mailing list